Open
Bug 1326647
Opened 7 years ago
Updated 2 years ago
Page inconsistently scrolls to the beginning horizontally and vertically when I focus it
Categories
(Core :: DOM: UI Events & Focus Handling, defect, P3)
Core
DOM: UI Events & Focus Handling
Tracking
()
People
(Reporter: arni2033, Unassigned)
References
()
Details
Attachments
(1 file)
3.96 KB,
text/plain
|
Details |
>>> My Info: Win7_64, Nightly 49, 32bit, ID 20160526082509 STR_1: 1. Open url data:text/html,<a href="http://a.b">Link</a><style>body{height:10000px;width:10000px} 2. Scroll the page by 2 screens to the bottom and a by 2 screens to the right 3. Click in searchbar or press Ctrl+K 4. Press Tab or F6 to focus the page AR: The page is scrolled to the beginning (only horizontally) ER: The page shouldn't scroll STR_2: (reference of good behavior; difference between STR_1 and STR_2 is only Step 2) 1. Open url data:text/html,<a href="http://a.b">Link</a><style>body{height:10000px;width:10000px} 2. Scroll the page by ~100px to the bottom and a by ~100px to the right 3. Click in searchbar or press Ctrl+K 4. Press Tab or F6 to focus the page AR: The page doesn't scroll anyhow, just as expected STR_3: 1. Log in on https://vk.com/ 2. Open https://vk.com/im, start a conversation with somebody (e.g. yourself: https://vk.com/im?sel=<your_id> ) 3. Write a lot of comments, at least 10 screens in height 4. Reload the page 5. Click in urlbar 6. Press F6 (or Tab several times, depending on the new buggy searchbar state) to focus content area AR: The page is scrolled to beginning (vertically); it causes the site to load lots of old messages ER: The page should stay still; the browser shouldn't cause site to load old messages STR_4: (testcase) 1. Open url [1], press PageDown twice to scroll the page 2. Click in urlbar 3. Press F6 (or Tab several times, depending on the new buggy searchbar state) to focus content area AR: The page is scrolled to the very beginning (vertically) ER: The page should stay still > [1] data:text/html,<div class="d0"><div class="d1"><div class="d2"><style>body{height:100%}.d0{height:10000px;}.d1{position:fixed;top:50px;height:150px;left:50px;width:150px;background:lightgray;}.d2{position:absolute;top:50px;height:50px;left:50px;width:50px;background:black;} Note: Actually, expectations are "either never scroll the page, or always scroll it" I haven't mentioned the latter, because it's just a bad UI.
Comment 1•7 years ago
|
||
User Agent Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:53.0) Gecko/20100101 Firefox/53.0 Build ID 20170115030210 The issue is reproducible on the latest Firefox release (50.1.0) and the latest Nightly (53.0a1).
status-firefox50:
--- → affected
status-firefox51:
--- → affected
status-firefox52:
--- → affected
status-firefox53:
--- → affected
Component: Untriaged → Panning and Zooming
Comment 2•7 years ago
|
||
I got a backtrace on OS X of the call to the ScrollToWithOrigin that's shifting the scroll position back to x=0. Note that this happens with or without APZ, it seems to be focus-related.
Comment 3•7 years ago
|
||
I'm assuming the call to MoveFocus at [1] might need tweaking, and that was added in bug 1176239 by Neil. [1] http://searchfox.org/mozilla-central/rev/790b2cb423ea1ecb5746722d51633caec9bab95a/dom/ipc/TabChild.cpp#2453
Component: Panning and Zooming → DOM: Events
Comment 4•7 years ago
|
||
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #3) > I'm assuming the call to MoveFocus at [1] might need tweaking, and that was > added in bug 1176239 by Neil. > > [1] > http://searchfox.org/mozilla-central/rev/ > 790b2cb423ea1ecb5746722d51633caec9bab95a/dom/ipc/TabChild.cpp#2453 hello Neil, do you have ideas of what we shall do here? Thanks.
Flags: needinfo?(enndeakin)
Comment 5•7 years ago
|
||
I guess what we would want to do is when a root node of a document is being focused via keyboard navigation, scroll the parent frame into view instead of that node. That should also not regress bug 1328043.
Flags: needinfo?(enndeakin)
Updated•7 years ago
|
Component: DOM: Events → Keyboard: Navigation
Priority: -- → P2
Comment 6•6 years ago
|
||
Moving to p3 because no activity for at least 1 year(s). See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Priority: P2 → P3
Assignee | ||
Updated•5 years ago
|
Component: Keyboard: Navigation → User events and focus handling
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•