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
264702
Handle empty title attribute tooltips
https://bugs.webkit.org/show_bug.cgi?id=264702
Summary
Handle empty title attribute tooltips
Ahmad Saleem
Reported
2023-11-12 05:18:44 PST
Hi Team, While going through Blink's commit, I came across another potential merge and to fix the bugs. Blink Commit:
https://chromium.googlesource.com/chromium/blink/+/a57f7f4625cd5d699a41154c67436ae423f7a5e1
WebKit Source:
https://searchfox.org/wubkat/rev/6e975c79b0d4766d80b0421a6dedeae64ca84b63/Source/WebCore/rendering/HitTestResult.cpp#281
Test Case:
https://jsfiddle.net/2fsd1La3/
^ hover on 'PageTitle' and you will get tooltip while if you hover on 'span element..', in STP182, you will get parent's tooltip. While Chrome Canary 121 and Firefox Nightly 121 does not show parent's tooltip. Just wanted to raise and fix it. Thanks!
Attachments
Add attachment
proposed patch, testcase, etc.
Ahmad Saleem
Comment 1
2023-11-12 05:21:14 PST
Web-Spec:
https://html.spec.whatwg.org/#the-title-attribute
The advisory information of an element is the value that the following algorithm returns, with the algorithm being aborted once a value is returned. When the algorithm returns the empty string, then there is no advisory information. If the element has a title attribute, then return its value. If the element has a parent element, then return the parent element's advisory information. Return the empty string.
Ahmad Saleem
Comment 2
2023-11-16 01:13:01 PST
@Aditya - do we have any 'test mechanism' for 'tooltips' like Blink / Chromium in WebKit? The attached test from PR or from source.chromium.org does not work 1-1 in WebKit and also I checked WPT, if we have any similar test (which we might be failing etc.). In local testing, this PR does fix the issue in Test Case from
Comment 0
.
Radar WebKit Bug Importer
Comment 3
2023-11-19 05:19:14 PST
<
rdar://problem/118619793
>
Ahmad Saleem
Comment 4
2024-01-04 15:31:15 PST
PR -
https://github.com/WebKit/WebKit/pull/22403
Manage to make test. Yuppee!!
EWS
Comment 5
2024-01-04 18:50:02 PST
Committed
272667@main
(e7d1f5b567cb): <
https://commits.webkit.org/272667@main
> Reviewed commits have been landed. Closing PR #22403 and removing active labels.
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