Closed Bug 391541 Opened 13 years ago Closed 12 years ago

Scrollbars do not change appearance on implict hover

Categories

(Core :: Widget: Win32, defect)

x86
Windows Vista
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: robarnold, Assigned: robarnold)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Most widgets have a normal appear, hover/hot appearance and a pressed appearance. On Vista however, there is an addition 'implicit' hover state for the scrollbar buttons whenever the mouse is over any part of the scrollbar control.

I could not find a theme part/state combintation that would give the desired appearance for this state. Perhaps we should look into using images to fake it like we used to for menus.
There are 4 new states for the theme part for scrollbar buttons, these additional states are this 'implicit hover'. (I'd confirm what the exact numbers are, but I'm too far detached from my home machine, which has my theme browser on.)
So, the scrollbar, arrow button part (1) has the following states:

- 1-4 up button (normal, hot, pressed, disabled)
- 5-8 down button (normal, hot, pressed, disabled)
- 9-12 left button (normal, hot, pressed, disabled)
- 13-16 right button (normal, hot, pressed, disabled)
- 17 up button (implicit hover)
- 18 down button (implicit hover)
- 19 left button (implicit hover)
- 20 right button (implicit hover)

The last 4 are the new, Vista ones. These are the ones you'll need for fixing this bug, although getting the rendering code to actually do the implict bit might be quite fun.
Status: NEW → ASSIGNED
Assignee: nobody → robarnold
Status: ASSIGNED → NEW
Attachment #276551 - Flags: review?(vladimir)
Comment on attachment 276551 [details] [diff] [review]
Restores native behavior, modulo animation

Looks fine, but can you put in a comment saying what that magic 17 is or something?
Attachment #276551 - Flags: review?(vladimir) → review+
Blocks: 373266
Status: NEW → ASSIGNED
new revision: 3.90; previous revision: 3.89
done
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Depends on: 393241
You need to log in before you can comment on or make changes to this bug.