1906895 - Text-Fragments: Scroll the start container of the range into view instead of its common ancestor

Status: RESOLVED FIXED

(Core :: DOM: Navigation, task)

Description

[Jan Jaeschke [:jjaschke]]

Attachment: closest-ancestor.html

Currently, the spec requires the closest common ancestor to be scrolled to, which is currently implemented in Gecko. This can lead to unwanted results if the common ancestor is not in the same view port as the text directive range (see example). Therefore, it makes more sense to scroll to the start container of the text directive range.

Comment 1

[Jan Jaeschke [:jjaschke]]

Link to attached file with text fragment.

Comment 2

[Jan Jaeschke [:jjaschke]]

Spec issue.

Comment 3

[Jan Jaeschke [:jjaschke]]

Attachment: Bug 1906895 - Text Fragments: Scroll the start container of a range into view instead of its common ancestor. r=emilio!,#dom-core

Spec issue: https://github.com/WICG/scroll-to-text-fragment/issues/259

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/91c93776446c
Text Fragments: Scroll the start container of a range into view instead of its common ancestor. r=emilio

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/47125 for changes under testing/web-platform/tests

Comment 6

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/91c93776446c

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot