Bug 180872

Summary:

Erroneous telephone number replacement breaks this content

Product:

WebKit

Reporter:

Simon Fraser (smfr) <simon.fraser>

Component:

WebKit Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

NEW

Severity:

Normal

CC:

dbates, simon.fraser, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

Safari 11

Hardware:

Unspecified

OS:

Unspecified

URL:

http://eucomp.hu/ipad3/test.html

Attachments:

Description	Flags
Testcase	none

Simon Fraser (smfr)

Reported 2017-12-15 11:05:58 PST

Load this page in Safari on macOS, and Safari on iPad: http://eucomp.hu/ipad3/test.html Note how the 5th column of the table on iPad is empty, and the numbers appear overlapping the first column. This is triggered by telephone number replacement.

Attachments
Testcase (873 bytes, text/html) 2017-12-15 11:20 PST, Simon Fraser (smfr)	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2017-12-15 11:06:49 PST

<rdar://problem/36077319>

Simon Fraser (smfr)

Comment 2 2017-12-15 11:20:45 PST

Created attachment 329500 [details] Testcase

Simon Fraser (smfr)

Comment 3 2017-12-15 11:39:48 PST

The problem seems to be that there's an enclosing <a>, and we close that <a> before inserting the tel: link because you can't test <a>s. I think we should just not convert to telephone numbers if there's an <a> ancestor.

Note You need to log in before you can comment on or make changes to this bug.