|Summary:||[GTK] Scrollbars look bad with GTK+ 3.16|
|Product:||WebKit||Reporter:||Michael Catanzaro <mcatanzaro>|
|Severity:||Normal||CC:||cgarcia, gustavo, mcatanzaro, pnormand, svillar, yoseforb|
|Version:||528+ (Nightly build)|
Description Michael Catanzaro 2015-01-22 16:29:34 PST
Created attachment 245181 [details] Screenshot: upstream scrollbars are half this wide and only appear on mouseover GTK+ has started using overlay scrollbars (at least with the default Adwaita theme). Our scrollbars look out of place, and are too wide.
Comment 1 Michael Catanzaro 2015-03-04 07:08:03 PST
I think the cross-platform code already supports overlay scrollbars. Does not look simple to hook up, though.
Comment 2 Carlos Garcia Campos 2015-03-04 07:14:42 PST
Even if we don't use the overlay scrollbars, the scrollbars should look good, something has changed in adwaita once again that broke our theming code.
Comment 4 Sergio Villar Senin 2015-03-19 03:03:12 PDT
Comment on attachment 249028 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=249028&action=review r+ with a nit I guess we haven't detected it in the bots because they don't have 3.16. It's unfortunate to have to depend on dogfeeding to detect this kind of regressions. > Source/WebCore/platform/gtk/ScrollbarThemeGtk.cpp:281 > +static void adjustRectAccoringToMargin(GtkStyleContext* context, GtkStateFlags state, IntRect& rect) According. Fix also Changelogs Also perhaps adjustScrollbarRectToMargin?
Comment 5 Carlos Garcia Campos 2015-03-19 03:06:50 PDT
(In reply to comment #4) > Comment on attachment 249028 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=249028&action=review > > r+ with a nit Thanks for the review. > I guess we haven't detected it in the bots because they don't have 3.16. > It's unfortunate to have to depend on dogfeeding to detect this kind of > regressions. > > > Source/WebCore/platform/gtk/ScrollbarThemeGtk.cpp:281 > > +static void adjustRectAccoringToMargin(GtkStyleContext* context, GtkStateFlags state, IntRect& rect) > > According. Fix also Changelogs Oops. > Also perhaps adjustScrollbarRectToMargin? The method is more generic than that, it's used for scrollbars, but has nothing to do with scrollbars.