Stop styling <select multiple> like buttons in non-native theme
Categories
(Core :: Widget, defect, P2)
Tracking
()
People
(Reporter: emilio, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
image/png
|
Details |
This uncovered a test bug in bug 1621319, but I think I agree with markus on the review that there's a lot of padding.
Updated•5 years ago
|
Comment 1•5 years ago
|
||
I think we should just stop styling <select multiple> like we style buttons... It shouldn't have any hover or active state or padding, like listboxes on most platforms.
Updated•5 years ago
|
Comment 2•5 years ago
|
||
Comment 3•5 years ago
|
||
I haven't requested review because the patch needs a rebase, and also because using the PaintTextField function isn't particularly great (even if it gets us closer to looking like a listbox).
Updated•4 years ago
|
Comment 4•4 years ago
|
||
This has been fixed elsewhere I think.
Emilio, can you confirm if your original bug was fixed and close this? Thanks!
Reporter | ||
Comment 5•4 years ago
|
||
Yeah, this was fixed in bug 1640195.
Comment 6•4 years ago
|
||
I think this issue is still reproducible, see the attached screenshot (select-multiple-arrow-hover-state.png). I tested on Windows 8.1 x64 and Windows 10 x64, with pref widget.disable-native-theme-for-content = true, on Fx 86.0 and on latest nightly, access bugzilla.com website/Advanced Search for example on Classification or Status multi selection listboxes the scroll have hover state and activ state too. If the issue is not about this bug please ping me and I file a new bug. Thanks!
Reporter | ||
Comment 7•4 years ago
|
||
No, that's the scrollbar arrow, which is expected to have hover styles. Before we used to apply the hover style to the whole listbox.
Description
•