Open Bug 1743602 Opened 2 years ago Updated 2 years ago

Non-scrollable page can hide dynamic toolbar

Categories

(Core :: Panning and Zooming, defect, P3)

defect

Tracking

()

People

(Reporter: tnikkel, Unassigned)

References

Details

Load https://neartalk.makeworld.space/ and scroll up and down. You can show/hide the dynamic toolbar, but the page is not scrollable so the dynamic toolbar should always be shown.

This seems sensitive to at least the dev pixel ratio and maybe other things. I can consistently reproduce when running a local x86 build in the emulator, it has a dev pixel ratio of 2. On a real phone with dev pixel ratio of ~2.6 I do not see the problem.

(This came up in bug 1738696 where I can only reproduce the original bug when we correctly always show the dynamic toolbar.)

Bug 1652190 initially implemented this I think. One thing I see from that bug is this line

https://searchfox.org/mozilla-central/rev/4c184ca81b28f1ccffbfd08f465709b95bcb4aa1/layout/base/nsLayoutUtils.cpp#8111

should potentially be a fuzzy comparison. What I mean is that sometimes due to rounding issues the composition bounds are slightly smaller than the scrollable rect, but in reality they are the same size.

See Also: → 1652190

Would definitely be nice to fix but the symptoms are not particularly severe so going with S3.

Severity: -- → S3
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.