Summary: | <use> cursor not changing (to hand) on hover | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | jay <jay> | ||||||||
Component: | SVG | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | zimmermann | ||||||||
Priority: | P2 | ||||||||||
Version: | 420+ | ||||||||||
Hardware: | Mac | ||||||||||
OS: | OS X 10.4 | ||||||||||
URL: | http://www.peepo.co.uk | ||||||||||
Attachments: |
|
Description
jay
2007-02-06 01:34:41 PST
Created attachment 12962 [details]
look: No Hands
Created attachment 12967 [details]
A much nicer test case
A few things were wrong with the original test case: 1. it was uploaded as image/svg-xml. That's incorrect. The final SVG 1.1 spec uses image/svg+xml. 2. it used fill='eee', that worked OK in Safari (and would have in IE) but it isn't valid by the SVG spec. fill='#eee' would have been correct. bugzilla autodetect for mozilla and safari uploads as image/svg-xml The patch attached to bug 12580 is a step in the right direction. However it's still not quite fixed. This shouldn't be hard to fix, but I should probably speak with WildFox first. That's odd. When I upload from Safari, bugzilla correctly autodetects image/svg+xml likely Opera... (In reply to comment #7) > likely Opera... Usually the operating system plays a role in file types as well. In Windows, it's the registry. In Linux, it's /etc/mime.types. Created attachment 14824 [details]
fix and manual test
I'm pretty certain this is the correct fix. The only thing which could possibly be wrong is if Layer::hitTest has some caller which wouldn't want to know the URLElement ignoring shadow nodes. Such a caller seems highly unlikely since URLElement is only used by link hover code AFAIK.
Landed as r21937 on the feature-branch. Yet another change which probably should move to TOT. |