WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
258622
Selection should not set the cursor type to text over the explicitly set cursor type
https://bugs.webkit.org/show_bug.cgi?id=258622
Summary
Selection should not set the cursor type to text over the explicitly set curs...
Ahmad Saleem
Reported
2023-06-28 06:43:51 PDT
Hi Team, While going through Blink's commit, I came across another failing test in Safari Technology Preview 172 and WebKit ToT while passing in Chrome Canary 117 and Firefox Nightly 116. Test Case:
https://jsfiddle.net/qe3dkgaw/show
^ Open and drag select text, Safari will have 'I-Beam' (text) cursor while others will have 'progress' cursor. Blink Commit:
https://chromium.googlesource.com/chromium/blink/+/a5c471c481544403615780bce2845de58e37e341%5E%21/#F2
WebKit Source:
https://searchfox.org/wubkat/source/Source/WebCore/page/EventHandler.cpp#1555
This commit does not modify single thing but just pushes the 'if' below 'switch' case. In local testing, now we pass attached test case at least in "Minibrowser" and get 'progress' cursor. Thanks!
Attachments
Add attachment
proposed patch, testcase, etc.
EWS
Comment 1
2023-06-28 13:46:49 PDT
Committed
265597@main
(15042015b0c8): <
https://commits.webkit.org/265597@main
> Reviewed commits have been landed. Closing PR #15372 and removing active labels.
Radar WebKit Bug Importer
Comment 2
2023-06-28 13:47:17 PDT
<
rdar://problem/111469521
>
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