Closed Bug 739289 Opened 13 years ago Closed 13 years ago

Far too much space at bottom of page with position fixed elements when zoomed in

Categories

(Firefox for Android Graveyard :: General, defect)

ARM
Android
defect
Not set
normal

Tracking

(blocking-fennec1.0 +)

RESOLVED WORKSFORME
Tracking Status
blocking-fennec1.0 --- +

People

(Reporter: martijn.martijn, Assigned: kats)

References

()

Details

(Whiteboard: [gfx])

Attachments

(2 files)

Attached image Screenshot bottom of page zoomed out (deleted) —
I found this while investigating bug 739053. When zooming in, there is far too much 'free space' at the bottom, which you don't get while zoomed in. See screenshots. This is with the latest trunk build on the Samsung Galaxy SII.
Attached image Screenshot bottom of page zoomed in (deleted) —
This is the screenshot that shows the bug in question.
There is a position:fixed element on the page that is screwing with us. When you zoom in, it gets positioned below the bottom of the viewport, and somehow causes the page size to increase (presumably it affects the mContentSize in the compositor). This is tricky, I don't know how we're supposed to be dealing with position:fixed elements.
blocking-fennec1.0: --- → ?
also adding a dependency to bug 732016 because it looks vaguely related.
Assignee: nobody → bugmail.mozilla
blocking-fennec1.0: ? → beta+
Depends on: 732016
Summary: Far too much space at bottom of this page when zoomed in → Far too much space at bottom of page with position fixed elements when zoomed in
Whiteboard: [gfx]
blocking-fennec1.0: beta+ → +
please comment out this line [https://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/browser.js#1560] and retest. If that fixes it, then this is a dupe of bug 732016.
Keywords: qawanted
Unfortunately, I can't reproduce this anymore on the url in question, I even tried this on a build at the time I filed this bug. I guess something in the page has changed. I tried to get a testcase based on comment 3, but I haven't been able to get position:fixed elements to influence the size of the viewport at all. Perhaps there is some relation to bug 656036 here? Perhaps any of the devs know how to reproduce this, still?
Built a debug with clamp scroll position commented out and it reintroduced this issue; as per comment #4. Tested on Nightly proper and I could not reproduce.
Keywords: qawanted
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WORKSFORME
(In reply to Aaron Train [:aaronmt] from comment #6) Comment 4 is saying that commenting out that line would fix it, not reintroduce it. So is this still a dupe? Aaron, could you perhaps make a minimized testcase out of the issue?
I still believe that the original problem on this page was due to position:fixed elements and is unrelated to bug 732016. If it is no longer happening that's fine, we can leave this bug as WFM. There's an endless supply of pages on which this will be reproducible (and there are already bugs open for some of them) so we'll fix it eventually.
No longer depends on: 732016
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: