Opened 7 years ago

Closed 7 years ago

#13805 closed enhancement (fixed)

HaikuDepot: Keep window position when browsing screenshots

Reported by: humdinger Owned by: stippi
Priority: normal Milestone: Unscheduled
Component: Applications/HaikuDepot Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

This is hrev51614.

If there are several screenshots with different sizes for an app, the window jumps around when browsing through them. Instead of centering the screenshot window with the screen (or is it the HaikuDepot main window? I'm on an erlier hrev ATM...), it'd be nicer if the top left corner of the window stayed the same. Then you can rest your mouse on the "Next" button, and just click through all shots.

Change History (5)

comment:1 by jua, 7 years ago

This is a somewhat tricky issue. I generally don't like that the window is being resized when flicking through screenshots, it's distracting. So I'd like to get rid of that entirely. But how to do it is still an open issue. An idea is to pre-load all screenshots in the background and then determine the maximum size and use that troughout... but loading all screenshots before displaying one would take too long.

comment:2 by humdinger, 7 years ago

I guess having the HDS server provide that info, maybe even including thumbnails for previewing, would complicate things?

comment:3 by jua, 7 years ago

On the HaikuDepot side it would make things easy, not sure though about the server side...

comment:4 by jua, 7 years ago

Andrew let me know that the server side already gathers this information, and it's even included in the JSON files and parsed in HaikuDepot into ScreenshotInfo::Width() and Height(). I completely overlooked that before!

So yeah, this is doable.

comment:5 by jua, 7 years ago

Resolution: fixed
Status: newclosed

Implemented in hrev51625.

One TODO left: it's still possible for the screenshot window to resize itself to be large than the screen size, this needs to be limited. However, then the bitmap view should also become scrollable, which needs to be implemented.

Note: See TracTickets for help on using tickets.