Closed
Bug 811301
Opened 12 years ago
Closed 12 years ago
setting html {overflow:hidden} resets window scroll position
Categories
(Core :: Layout, defect)
Tracking
()
RESOLVED
FIXED
mozilla19
People
(Reporter: zack, Assigned: roc)
References
(Depends on 1 open bug)
Details
Attachments
(4 files)
245 bytes,
text/html
|
Details | |
4.93 KB,
patch
|
MatsPalmgren_bugz
:
review+
|
Details | Diff | Splinter Review |
16.64 KB,
patch
|
MatsPalmgren_bugz
:
review+
|
Details | Diff | Splinter Review |
6.38 KB,
patch
|
MatsPalmgren_bugz
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_7_4) AppleWebKit/537.11 (KHTML, like Gecko) Chrome/23.0.1271.64 Safari/537.11 Steps to reproduce: Firefox v16.0.2 to reproduce: - set (with css) html {overflow-y:scroll} because we want to force vertical scroll bars - set (with js) style="overflow:hidden;" (also tested style="overflow-y:hidden;") on the html element, because we are opening a dialog and want to disable scrolling. - firefox resets vertical window scroll position to top, and window.scrollTop to 0. (And does not fire a window.scroll event - although I wouldnt necessarily expect the event, since I believe this behaviour to be a bug.) scroll position should not change. for correct behaviour see IE8, IE9, Chrome (latest: 23.0.x), Safari (latest: 6.0) Actual results: - firefox resets vertical window scroll position to top, and window.scrollTop to 0. (And does not fire a window.scroll event - although I wouldnt necessarily expect the event, since I believe this behaviour to be a bug.) Expected results: scroll position should not change. for correct behaviour see IE8, IE9, Chrome (latest: 23.0.x), Safari (latest: 6.0)
Comment 1•12 years ago
|
||
Could you provide a minimized test case?
Component: Untriaged → Layout
Product: Firefox → Core
seems firefox has some well know bugs with html {overflow-y:scroll} https://github.com/h5bp/html5-boilerplate/issues/918 https://github.com/necolas/normalize.css/issues/71 https://github.com/necolas/normalize.css/issues/65 https://github.com/necolas/normalize.css/issues/64
Comment 3•12 years ago
|
||
Robert, should be be resetting scroll position to 0 when switching to hidden? Or is it an artifact of the reframe?
Assignee | ||
Comment 4•12 years ago
|
||
I don't know.
Assignee | ||
Comment 5•12 years ago
|
||
Assignee | ||
Comment 6•12 years ago
|
||
Assignee: nobody → roc
Attachment #681220 -
Flags: review?(matspal)
Assignee | ||
Comment 7•12 years ago
|
||
Attachment #681221 -
Flags: review?(matspal)
Assignee | ||
Comment 8•12 years ago
|
||
Attachment #681225 -
Flags: review?(matspal)
Updated•12 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•12 years ago
|
Attachment #681220 -
Flags: review?(matspal) → review+
Updated•12 years ago
|
Attachment #681221 -
Flags: review?(matspal) → review+
Comment 9•12 years ago
|
||
Comment on attachment 681225 [details] [diff] [review] Part 3: Cleanup presshell code now that we don't do special state save/restore for the root scroll frame Nice cleanup.
Attachment #681225 -
Flags: review?(matspal) → review+
Assignee | ||
Comment 10•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/58a64dbc9e83 https://hg.mozilla.org/integration/mozilla-inbound/rev/3493741d8e8d https://hg.mozilla.org/integration/mozilla-inbound/rev/34a4de5feafe
Comment 11•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/58a64dbc9e83 https://hg.mozilla.org/mozilla-central/rev/3493741d8e8d https://hg.mozilla.org/mozilla-central/rev/34a4de5feafe
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in
before you can comment on or make changes to this bug.
Description
•