After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 747827 - firefox scrollbars broken
firefox scrollbars broken
Status: RESOLVED NOTGNOME
Product: gtk+
Classification: Platform
Component: Themes
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2015-04-14 07:57 UTC by Jakub Steiner
Modified: 2015-04-15 12:57 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
scrollbar sliders have no margin (6.29 KB, image/png)
2015-04-14 07:57 UTC, Jakub Steiner 		  Details
how firefox scrollbars look here (11.02 KB, image/png)
2015-04-14 22:50 UTC, Matthias Clasen 		  Details
HighContrast: don't need to work around ffox (1.60 KB, patch)
2015-04-15 12:56 UTC, Jakub Steiner 		none Details | Review

Description Jakub Steiner 2015-04-14 07:57:04 UTC 
Created attachment 301512 [details]
scrollbar sliders have no margin

This may as well be a bug in Firefox rather than gtk+, but I'd like to make sure we're doing our best before filing against Firefox. As the screenshot demonstrates, the scrollbar slider in Firefox 36 is lacking the margin.

I "fixed" the issue in the high contrast theme by using a barely opaque border (commit https://git.gnome.org/browse/gtk+/commit/?id=6c8790f62c3bc4ff3fa4451948be3406f5525c34)

Adwaita itself seems to be using actual margin for this, which is something I was surprised to see as that's somethingwe rarely get to enjoy.

Is this something we can fix in the theme or does Firefox need to do something particular?
Comment 1 Lapo Calamandrei 2015-04-14 11:41:13 UTC 
Firefox issue to me, chasing "alien" applications incompatibilities, by working them around in the theme doesn't seem to me a good thing to do.
Comment 2 Matthias Clasen 2015-04-14 22:50:42 UTC 
Created attachment 301578 [details]
how firefox scrollbars look here

Firefox scrollbars look less broken here.
Comment 3 Jakub Steiner 2015-04-15 12:56:37 UTC 
Created attachment 301625 [details] [review]
HighContrast: don't need to work around ffox

No need for the workaround for ffox 36, works in 37
Comment 4 Jakub Steiner 2015-04-15 12:57:23 UTC 
Seems to have been a bug in ffox that has been fixed in the meantime indeed. Reverted the workaround in HC