Bug 198378

Summary: [iOS] Do not linkify telephone numbers inside <a> elements.
Product: WebKit Reporter: alan <zalan>
Component: DOMAssignee: alan <zalan>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, commit-queue, esprehn+autocc, ews-watchlist, gyuyoung.kim, simon.fraser, webkit-bug-importer, wenson_hsieh
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

alan
Reported 2019-05-30 09:26:49 PDT
rdar://problem/43208329
Attachments
Patch (5.23 KB, patch)
2019-05-30 10:02 PDT, alan 		no flags
DetailsFormatted DiffDiff
Patch (5.32 KB, patch)
2019-05-30 11:05 PDT, alan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
alan
Comment 1 2019-05-30 10:02:58 PDT
Created attachment 370949 [details] Patch
Wenson Hsieh
Comment 2 2019-05-30 10:24:02 PDT
Could you double check that this doesn't break phone number data detection in Mail display?
alan
Comment 3 2019-05-30 10:36:26 PDT
(In reply to Wenson Hsieh from comment #2) > Could you double check that this doesn't break phone number data detection > in Mail display? Should be fine as long as the content is not wrapped inside an <a> element.
Wenson Hsieh
Comment 4 2019-05-30 10:36:57 PDT
(In reply to zalan from comment #3) > (In reply to Wenson Hsieh from comment #2) > > Could you double check that this doesn't break phone number data detection > > in Mail display? > Should be fine as long as the content is not wrapped inside an <a> element. 👍🏻
alan
Comment 5 2019-05-30 10:43:03 PDT
Apparently linkifying is off by default even on iOS (testing).
alan
Comment 6 2019-05-30 11:05:57 PDT
Created attachment 370957 [details] Patch
WebKit Commit Bot
Comment 7 2019-05-30 12:41:59 PDT
Comment on attachment 370957 [details] Patch Clearing flags on attachment: 370957 Committed r245900: <https://trac.webkit.org/changeset/245900>
WebKit Commit Bot
Comment 8 2019-05-30 12:42:01 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.