WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED CONFIGURATION CHANGED
245269
setBaseAndExtent not setting focus as requested
https://bugs.webkit.org/show_bug.cgi?id=245269
Summary
setBaseAndExtent not setting focus as requested
Jamie Birch
Reported
2022-09-16 01:36:29 PDT
Created
attachment 462386
[details]
HTML file reproducing the issue. STR: 1: Open attachment: setbaseandextent_not_setting_focus_as_requested_repro.html 2. Click the "Generate selection" button. Expected results: On-screen logger should report the resulting selection, which reflects the focusNode and focusOffset passed as parameters in the call to Selection.setBaseTextAndExtent(): { "focusNode": "#text:Text outside:", "focusOffset": 13, "anchorNode": "#text:_", "anchorOffset": 1 } Actual results: On-screen logger reports a different focusNode and focusOffset (the start of the text node from the neighbouring element): { "focusNode": "#text:_", "focusOffset": 0, "anchorNode": "#text:_", "anchorOffset": 1 } Firefox and Chrome produce the expected results.
Attachments
HTML file reproducing the issue.
(2.76 KB, text/html)
2022-09-16 01:36 PDT
,
Jamie Birch
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2022-09-21 20:44:54 PDT
<
rdar://problem/100254222
>
Karl Dubost
Comment 2
2022-09-21 21:08:54 PDT
Spec:
https://w3c.github.io/selection-api/#dom-selection-setbaseandextent
Tests:
https://wpt.fyi/results/selection/setBaseAndExtent.html?label=experimental&label=master&aligned
I wonder how reliable the tests are with regards to the current spec.
Ahmad Saleem
Comment 3
2023-03-14 16:58:48 PDT
I think Selection API ‘Live Range’ fixed it since now we are passing all WPT tests cases mentioned by
Comment 02
and also in the attached, I am gettign “Expected Result” using WebKit ToT and Safari Technology Preview 165. { "focusNode": "#text:Text outside:", "focusOffset": 13, "anchorNode": "#text:_", "anchorOffset": 1 }
Ryosuke Niwa
Comment 4
2023-03-14 21:49:28 PDT
Nice
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug