22500 2008-11-25 15:26:19 -0800 hyperlinked text parts aren't rendered 2011-05-17 07:35:19 -0700 1 1 1 Unclassified WebKit SVG 528+ (Nightly build) All All RESOLVED WORKSFORME http://kaioa.com/svg/webkit_linktest.svg P2 Normal --- 1 webkit webkit-unassigned darabos.daniel eric jay rwlbuis oldest_to_newest 100311 0 webkit 2008-11-25 15:26:19 -0800 The example file contains the text "foo link foo" where "link" is wrapped with an <a> element whose "xlink:href" attribute has the value "http://example.com". It basically looks like this: <text [...]> <tspan>foo <a xlink:href="http://example.com">link</a> foo</tspan> </text> -- WebKit-r38708 only renders "foo foo". The same goes for current versions of Chrome and Safari. -- Hyperlinking a whole <text> element does work fine though. E.g. the following works as it should: <a [...]><text [...]><tspan [...]>foo</tspan></text></a> While the specs don't allow a <tspan> as a child of <a>, #PCDATA is valid. See: http://www.w3.org/TR/SVG/linking.html#Links (#PCDATA is listed first in SVG.a.content) So something like the following should work: <text [...]><tspan [...]><a [...]>foo</a></tspan></text> And indeed it's valid and other browsers such as Firefox or Opera display it correctly and the link is also clickable (<tspan> as child of <a> also works there - even if it isn't valid). 107663 1 darabos.daniel 2009-01-30 03:26:26 -0800 I can confirm this bug. FF3 displays the link and it is clickable, Chrome does not display the link at all. 119226 2 eric 2009-04-28 17:21:10 -0700 <a> tags are not allowed inside <tspan> according to the SVG 1.1 DTD: <!ENTITY % SVG.TextContent.extra.class "" > <!ENTITY % SVG.TextContent.class "| %SVG.tspan.qname; | %SVG.tref.qname; | %SVG.textPath.qname; | %SVG.altGlyph.qname; %SVG.TextContent.extra.class;" > http://www.w3.org/TR/SVG11/svgdtd.html This looks like a bug in Firefox which we should file with mozilla. 119246 3 webkit 2009-04-28 18:36:47 -0700 Aren't you looking at the wrong thingy? http://www.w3.org/TR/SVG11/text.html#TSpanElement <!ENTITY % SVG.tspan.content "( #PCDATA | %SVG.tspan.qname; | %SVG.tref.qname; | %SVG.altGlyph.qname; | %SVG.animate.qname; | %SVG.set.qname; | %SVG.animateColor.qname; | %SVG.Description.class; %SVG.Hyperlink.class; %SVG.tspan.extra.content; )*" > There... %SVG.Hyperlink.class. 135161 4 jay 2009-07-28 04:41:57 -0700 The example given validates using the w3c validator, mozilla, batik and opera display the anchor content. as stated anchor in text is supported by webkit there is a clear use case as per the example. #3 suggests imho correctly that the DTD already states that tspan may include an anchor: <!ENTITY % SVG.tspan.content "( #PCDATA | %SVG.tspan.qname; | %SVG.tref.qname; | %SVG.altGlyph.qname; | %SVG.animate.qname; | %SVG.set.qname; | %SVG.animateColor.qname; | %SVG.Description.class; %SVG.Hyperlink.class; %SVG.tspan.extra.content; )*" > common-sense dictates that if the spec had required tspan not to contain anchor, this would be raised as an errata, and this bug could be raised with the working group. 135162 5 33615 jay 2009-07-28 04:45:53 -0700 Created attachment 33615 demonstrates text and tspan with anchors, tested on moz, opera, batik, and webkit 404892 6 rwlbuis 2011-05-16 19:06:49 -0700 I think this works nowadays. Cheers, Rob. 405042 7 webkit 2011-05-17 07:27:46 -0700 Yes, it does look indeed like it was fixed in the meantime. Thanks for taking a look, Rob. :) 405046 8 rwlbuis 2011-05-17 07:35:19 -0700 Hi Jos, (In reply to comment #7) > Yes, it does look indeed like it was fixed in the meantime. Thanks for taking a look, Rob. :) No problem :) I like to close old reports so we can concentrate on fixing what is still broken today :) Cheers, Rob. 33615 2009-07-28 04:45:53 -0700 2009-07-28 04:45:53 -0700 demonstrates text and tspan with anchors, tested on moz, opera, batik, and webkit moz-webkit-text-anchor-bug.svg image/svg+xml 847 jay PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiIHN0YW5kYWxvbmU9Im5vIj8+Cjwh RE9DVFlQRSBzdmcgUFVCTElDICItLy9XM0MvL0RURCBTVkcgMjAwMTA5MDQvL0VOIg0KICAgICJo dHRwOi8vd3d3LnczLm9yZy9UUi8yMDAxL1JFQy1TVkctMjAwMTA5MDQvRFREL3N2ZzEwLmR0ZCI+ CjxzdmcgeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnLzIwMDAvc3ZnIiB2ZXJzaW9uPSIxLjIiIGJh c2VQcm9maWxlPSJ0aW55IgogICAgIHhtbG5zOnhsaW5rPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5 L3hsaW5rIiB3aWR0aD0iNDAwIiBoZWlnaHQ9IjMwMCI+Cjx0aXRsZT5tb3ppbGxhIGFuZCA8L3Rp dGxlPgogPHRleHQgZm9udC1zaXplPSIwLjdlbSIgeD0iNDQiIHk9IjMwIiB0ZXh0LWFuY2hvcj0i c3RhcnQiIGZpbGw9ImJsYWNrIj4gPGEgeGxpbms6aHJlZj0iaHR0cDovL3d3dy5ob250ZS5ldSIg ZmlsbD0iYmx1ZSI+d2Via2l0IHRleHQtb25seSBhbmNob3IgY2hlY2s8L2E+PC90ZXh0PgogIDx0 ZXh0IGZvbnQtc2l6ZT0iMC43ZW0iIHg9IjQ0IiB5PSI1MCIgdGV4dC1hbmNob3I9InN0YXJ0IiBm aWxsPSJibGFjayI+CiAgPHRzcGFuICB4PSI0NCIgZHk9IjFlbSIgID5XZWJraXQgZHJvcHMgYW5j aG9yIGNvbnRlbnQgaW4gdHNwYW46IDxhIHhsaW5rOmhyZWY9Imh0dHA6Ly93d3cuaG9udGUuZXUi IGZpbGw9ImJsdWUiPnRvIGh0dHA6Ly93d3cuaG9udGUuZXU8L2E+ICBhbmQ8L3RzcGFuPgogIDx0 c3BhbiAgeD0iNDQiIGR5PSIxZW0iICA+TW96aWxsYSBpbnNlcnRzIGV4dHJhIGVtcHR5IGxpbmUg YWZ0ZXIgYW5jaG9yLCBzZWUgYWJvdmU8L3RzcGFuPgo8L3RleHQ+CgogCjwvc3ZnPg==