Bug 15522 - REGRESSION: Combination of client-side image map and <a> tag is not working properly
Summary: REGRESSION: Combination of client-side image map and <a> tag is not working p...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 523.x (Safari 3)
Hardware: PC OS X 10.4
: P1 Normal
Assignee: Beth Dakin
URL: http://testsuite.nokia-boston.com/con...
Keywords: HasReduction, InRadar, Regression
Depends on:
Blocks:
 
Reported: 2007-10-15 12:13 PDT by zalan
Modified: 2007-11-14 16:00 PST (History)
2 users (show)

See Also:

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+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description zalan 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.
Comment 1 Mark Rowe (bdash) 2007-10-15 12:23:41 PDT 
<rdar://problem/5540855>
Comment 2 David Kilzer (:ddkilzer) 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.
Comment 3 Beth Dakin 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...
Comment 4 Beth Dakin 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.
Comment 5 Beth Dakin 2007-11-14 15:24:59 PST 
Created attachment 17283 [details]
Allow <map> to have inline children as well as block
Comment 6 Dave Hyatt 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.
Comment 7 Dave Hyatt 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.
Comment 8 Beth Dakin 2007-11-14 15:49:39 PST 
Fixed it! Thanks, Dave. Fixed with r27800.
Comment 9 zalan 2007-11-14 16:00:09 PST 
wow! Thanks a lot! :) I'll apply it to our trunk based branch to test....