Created attachment 8488988 [details] Screenshot of the issue User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0 Build ID: 20140911151253 Steps to reproduce: 1.In Firefox 32 apply a full theme with skinned scrollbars (for example: FT DeepDark). 2.Zoom a page to certain percentage. The percentage varies between themes, screen resolutions, and dpi settings. For example: for FT DeepDark, 1920x1080, 125% dpi - the bug could be triggered at 150% zoom. 3.Scroll the page. Actual results: Scrollbar disappears upon scrolling Expected results: Scrollbar shouldn't disappear upon scrolling
I can reproduce on Windows7 Aero as well as Classic STR 0. Maximized browser on standard monitor(1280x1024) 1. Install the full theme https://addons.mozilla.org/ja/firefox/addon/ft-deepdark/ and restart 2. Open https://addons.mozilla.org/ja/firefox/addon/ft-deepdark/ page 3. Zoom out (Ctrl + -) 4. Scroll down (Press down arrow key / Mouse wheel / Dragging thumb of scroll bar etc.)
[Tracking Requested - why for this release]: Regression Regression window(m-i) Good: https://hg.mozilla.org/integration/mozilla-inbound/rev/fb7aec714906 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0 ID:20140528033652 Bad: https://hg.mozilla.org/integration/mozilla-inbound/rev/a0586a5ec2da Mozilla/5.0 (Windows NT 6.1; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0 ID:20140528035352 Pushlog: http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=fb7aec714906&tochange=a0586a5ec2da Regressed by: a0586a5ec2da Markus Stange — Bug 1009679 - Layerize the scrollbar thumb on desktop, too. r=roc
I can reproduce this in my Linux VM.
Created attachment 8490758 [details] [diff] [review] patch I don't really understand why this fixes it, but it's the right thing to do anyway.
And the reason that this bug didn't hit native scrollbars is that native scrollbars don't zoom along with the page, and thus are less likely to have fractional coordinates. There's bug 401213 about this discrepancy.
(In reply to Markus Stange [:mstange] from comment #7) > And the reason that this bug didn't hit native scrollbars is that native > scrollbars don't zoom along with the page, and thus are less likely to have > fractional coordinates. > There's bug 401213 about this discrepancy. There ought to be a way to lock the scrollbar width to desktop theme like native scrollbars regardless the non-native theme. This varying width and disappearing thumb are very annoying here with the shipped by default Modern theme: Mozilla/5.0 (X11; Linux i686; rv:32.0) Gecko/20100101 SeaMonkey/2.29; Build ID: 20140904220119
The patch seems small. If it's safe enough, would it make sense to consider uplifts, possibly up to beta 33, if applicable? It's a long time to wait till 35 makes it to the release channel, and the annoyance to those affected by it is more than minor IMO.
(In reply to Avi Halachmi (:avih) from comment #13) > The patch seems small. If it's safe enough, would it make sense to consider > uplifts, possibly up to beta 33, if applicable? > > It's a long time to wait till 35 makes it to the release channel, and the > annoyance to those affected by it is more than minor IMO. Totally agree. The sooner this patch gets into our browser, the sooner users would stop cursing =)
FWIW, I applied the patch to Firefox 32.02, compiled locally and it seems to fix the issue. Here's a try push which should have the builds available soon for testing: https://tbpl.mozilla.org/?tree=Try&rev=a8b2473b33ff
Comment on attachment 8490758 [details] [diff] [review] patch I'll let the drivers decide on which branches they want to take this patch. Approval Request Comment [Feature/regressing bug #]: regression from bug 1009679 [User impact if declined]: disappearing scrollbars [Describe test coverage new/current, TBPL]: none, and it's hard to write a test for this bug [Risks and why]: low, the patch is very conservative and restores as much of the pre-bug 1009679 behavior as possible [String/UUID change made/needed]: none
Comment on attachment 8490758 [details] [diff] [review] patch Happy to take it for 33 & 34. It is way too late for 32.
Fixed in Fx 33 beta 6, Aurora and Nightly builds.
(In reply to Aris from comment #21) > Fixed in Fx 33 beta 6, Aurora and Nightly builds. Aris sounds like you've verified this on all branches. Can you let us know on which Operating System this was?
I have tested this on Win7x64 using my "NewScrollbars" add-on with todays Fx 33b6 and yesterdays Aurora/Nightly builds.
Thanks for the info Aris! I've also been able to reproduce the original issue on Firefox 32.0.2, with steps from comment 1, on both Windows 7 x64 and Ubuntu 12.10 x86, and can confirm that the issue no longer shows on: - Firefox 33 Beta 6 - BuildID=20140922173023 - latest Firefox 34 Aurora - BuildID=20140923004002 - latest Firefox 35 Nightly - BuildID=20140923030204
(In reply to Markus Stange [:mstange] from comment #6) > Created attachment 8490758 [details] [diff] [review] > patch > > I don't really understand why this fixes it Andrew is fixing the actual underlying bug in bug 1169370.