Bug 15522

Summary: REGRESSION: Combination of client-side image map and <a> tag is not working properly
Product: WebKit Reporter: zalan <zalan>
Component: Layout and RenderingAssignee: Beth Dakin <bdakin>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, mitz
Priority: P1 Keywords: HasReduction, InRadar, Regression
Version: 523.x (Safari 3)   
Hardware: PC   
OS: OS X 10.4   
URL: http://testsuite.nokia-boston.com/content/html_imgmaps/imgmaps_name_anchor.html
Attachments:
Description Flags
Allow <map> to have inline children as well as block hyatt: review+

zalan
Reported 2007-10-15 12:13:45 PDT
Links are showing up as plain text when they are linked to image map with the coords attribute. It works fine on Tiger.
Attachments
Allow <map> to have inline children as well as block (55.87 KB, patch)
2007-11-14 15:24 PST, Beth Dakin 		hyatt: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Rowe (bdash)
Comment 1 2007-10-15 12:23:41 PDT
<rdar://problem/5540855>
David Kilzer (:ddkilzer)
Comment 2 2007-10-15 20:30:52 PDT
(In reply to comment #0) > Links are showing up as plain text when they are linked to image map with the > coords attribute. > > It works fine on Tiger. This test fails in the oldest WebKit svn nightly: r11976.
Beth Dakin
Comment 3 2007-11-14 13:09:02 PST
The Web Inspector actually reports errors claiming that <a> and <br> are not allowed inside <map>! That doesn't seem right...
Beth Dakin
Comment 4 2007-11-14 14:01:37 PST
I have a patch for this. I am creating a bunch of test cases before I post it, though.
Beth Dakin
Comment 5 2007-11-14 15:24:59 PST
Created attachment 17283 [details] Allow <map> to have inline children as well as block
Dave Hyatt
Comment 6 2007-11-14 15:44:32 PST
Comment on attachment 17283 [details] Allow <map> to have inline children as well as block Ditch the "imgTag" part since I assume that's inEithertagList.
Dave Hyatt
Comment 7 2007-11-14 15:44:58 PST
Comment on attachment 17283 [details] Allow <map> to have inline children as well as block r=me if you make that change.
Beth Dakin
Comment 8 2007-11-14 15:49:39 PST
Fixed it! Thanks, Dave. Fixed with r27800.
zalan
Comment 9 2007-11-14 16:00:09 PST
wow! Thanks a lot! :) I'll apply it to our trunk based branch to test....
Note You need to log in before you can comment on or make changes to this bug.