Opened 22 months ago

Last modified 22 months ago

#13807 new enhancement

HaikuDepot: Previous/Next screenshot buttons style

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

Description

This is hrev51614.

Since the mouse often rests on the "Next" button when browsing through several screenshots of an app, the tool tip is bound to pop up often. The buttons functions are also quite obvious, so I think the tool tips can be removed.

Why the mouse-over effect on those buttons? We don't do that normally in our GUI. These buttons should be buttons, standing tall and proud. :)

Change History (5)

comment:1 by humdinger, 22 months ago

Oh, and it'd be nice if a left-click anywhere on the screenshot would be the same as clicking "Next". Saves aiming for the button. I don't think we need another shortcut to move to the previous screenshot. Normally an app only has a handful of shots and it's quick enough to click a few more "Next" to arrive at the previous shot (or use the "Previous" button after all).

comment:2 by jua, 22 months ago

Why the mouse-over effect on those buttons? We don't do that normally in our GUI.

Oh, we do! These are just normal BToolBar buttons. See e.g. the toolbar in ShowImage doing the same. It only looks weird here (and I agree it's weird) because the buttons are text-only with no icon... but that is planned. Didn't find the time anymore at the sprint to add the icons to the buttons (planning to use the same arrows as in ShowImage) - that's why the commit message said that it's still rough and unfinished...

Agreed about the tool tips. I thought "can't hurt to have them", but didn't consider the case of hovering over the buttons for a longer time to click through the list.

comment:3 by jua, 22 months ago

Exchanged the button texts for icons in hrev51625.

Oh, and it'd be nice if a left-click anywhere on the screenshot would be the same as clicking "Next".

My original idea on this was actually that clicking anywhere on the screenshot closes the window (the close button is an even smaller click target than the "next" button after all, and most packages only have one screenshot). But I find mapping it to "Next" very reasonable as well.

Any opinions on what would be better?

comment:4 by diver, 22 months ago

BTW, does left/right arrows work in that window? (Can't check right now).

comment:5 by humdinger, 22 months ago

My original idea on this was actually that clicking anywhere on the screenshot closes the window

I first thought, "hmm, that might be a bit unexpected". But now I think that it's actually neatly symmetric to opening the window by clicking on the screenshot. So,fine by me. :)

Note: See TracTickets for help on using tickets.