Issue metadata
Sign in to add a comment
|
Sequential Focus Navigation Start Point should always follow element programmatically scrolled into view |
||||||||||||||||||||||||
Issue descriptionVersion: m50 What steps will reproduce the problem? (1) Navigate to https://html.spec.whatwg.org/multipage/webappapis.html#webappapis (2) Search for "responsible browsing context". The first result should be scrolled to the top of the page: https://html.spec.whatwg.org/multipage/webappapis.html#responsible-browsing-context (3) Note that there is a link in the paragraph immediately following, which it would be convenient to tab to. (4) Hit <escape> to close the find in page dialog; page remains scrolled to the same element. (5) Hit <tab> to attempt to move focus to link. What is the expected output? What do you see instead? Expected: focus moves to link. Actual: focus moves to the first focusable element after the fragment which was initially navigated to. I would propose that scrollIntoView() should also cause the start point to be set. cf. bug 454172
,
Mar 18 2016
The proposal is about JS API Element.scrollIntoView() or some Blink's internal function scrollIntoView(), or both? The find-in-page widget runs some native code to get the search text selected and scroll into view and does not use JS API directly, though this is an implementation detail. Probably we want both.
,
Mar 18 2016
Related discussion https://github.com/whatwg/html/issues/834#issuecomment-194017739
,
Mar 22 2016
,
Mar 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0bb887b20c70582eeafad2a58ac1650b7159f2b6 commit 0bb887b20c70582eeafad2a58ac1650b7159f2b6 Author: kochi <kochi@chromium.org> Date: Tue Mar 22 07:04:17 2016 Set sequential focus navigation starting point on scrollIntoView() BUG= 594613 Review URL: https://codereview.chromium.org/1810293002 Cr-Commit-Position: refs/heads/master@{#382522} [modify] https://crrev.com/0bb887b20c70582eeafad2a58ac1650b7159f2b6/third_party/WebKit/LayoutTests/fast/events/sequential-focus-navigation-starting-point.html [modify] https://crrev.com/0bb887b20c70582eeafad2a58ac1650b7159f2b6/third_party/WebKit/Source/core/dom/Element.cpp [modify] https://crrev.com/0bb887b20c70582eeafad2a58ac1650b7159f2b6/third_party/WebKit/Source/core/editing/Editor.cpp
,
Mar 23 2016
,
Sep 29 2017
,
Sep 29 2017
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by tkent@chromium.org
, Mar 14 2016