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)
Tracking
(blocking-fennec1.0 +)
RESOLVED
WORKSFORME
Tracking | Status | |
---|---|---|
blocking-fennec1.0 | --- | + |
People
(Reporter: martijn.martijn, Assigned: kats)
References
()
Details
(Whiteboard: [gfx])
Attachments
(2 files)
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.
Reporter | ||
Comment 1•13 years ago
|
||
This is the screenshot that shows the bug in question.
Assignee | ||
Comment 2•13 years ago
|
||
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: --- → ?
Comment 3•13 years ago
|
||
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
Assignee | ||
Updated•13 years ago
|
Whiteboard: [gfx]
Updated•13 years ago
|
blocking-fennec1.0: beta+ → +
Comment 4•13 years ago
|
||
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
Reporter | ||
Comment 5•13 years ago
|
||
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?
Comment 6•13 years ago
|
||
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
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WORKSFORME
Reporter | ||
Comment 7•13 years ago
|
||
(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?
Assignee | ||
Comment 8•13 years ago
|
||
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.
Assignee | ||
Updated•13 years ago
|
Blocks: omtclayerbugs
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•