| Summary: | SVG Links do not show status text on hover | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||||
| Component: | SVG | Assignee: | Nobody <webkit-unassigned> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | ||||||||
| Priority: | P2 | ||||||||
| Version: | 420+ | ||||||||
| Hardware: | Mac | ||||||||
| OS: | OS X 10.4 | ||||||||
| URL: | http://www.w3.org/Graphics/SVG/Test/20061213/htmlEmbedHarness/full-linking-a-04-t.html | ||||||||
| Attachments: |
|
||||||||
|
Description
Eric Seidel (no email)
2007-02-04 03:43:41 PST
Created attachment 12914 [details]
The fix (and test case)
It's a little ugly to add all of these SVG_SUPPORTs into this base code, but it's following the existing practice from HTML (which special cases various HTML elements).
Created attachment 12915 [details]
even cleaner patch (and test case)
Comment on attachment 12915 [details]
even cleaner patch (and test case)
+#ifdef SVG_SUPPORT
+ else if (m_innerURLElement->hasTagName(SVGNames::aTag))
+ return m_innerURLElement->isLink();
+#endif
No need for else here. The if before ends in a return.
Would be nice if this had a layout test. We could easily add logging of the delegate methods used for status text.
r=me
|