Closed Bug 1073924 Opened 9 years ago Closed 9 years ago

Hovering over links in SVG don't cause cursor to change


(Core :: SVG, defect)

32 Branch
Not set



Tracking Status
firefox32 --- wontfix
firefox33 + fixed
firefox34 --- fixed
firefox35 --- fixed
firefox-esr31 --- unaffected


(Reporter: keith.s.wilkinson, Assigned: longsonr)



(Keywords: regression)


(2 files)

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0
Build ID: 20140923175406

Steps to reproduce:

1. Display 
    or drag and drop this SVG file onto Firefox. 
2. Hover over the link in top left corner. 

Actual results:

Cursor does not change to indicate that the link is clickable. 

Expected results:

Works as expected in Internet Explorer, but I don't know what the SVG spec. says.
Regression range:

Maybe one of these bugs about removing style sheet on-demand for SVG documents.
Keywords: regression
Regression window(m-i)
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0 ID:20140524122823
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0 ID:20140524130923

Regressed by: Bug 1013936
Blocks: 1013936
Ever confirmed: true
Attached patch pointer.txtSplinter Review
I did think about creating a common.css and loading that always. Would you have preferred that?
Attachment #8496580 - Flags: review?(jwatt)
Assignee: nobody → longsonr
Drive-by nit: Might be nice to add/extend a comment to make it clearer that these rules are expected to exactly match the corresponding rules in ua.css (and maybe have a comment in ua.css saying the same).

That way, if we have to update these rules in one spot for some reason, we'll know that we should probably also update the other spot.
OS: Windows 7 → All
Hardware: x86_64 → All
Comment on attachment 8496580 [details] [diff] [review]

Review of attachment 8496580 [details] [diff] [review]:


::: layout/svg/svg.css
@@ +75,5 @@
>    opacity: inherit;
>  }
> +/* nsDocumentViewer::CreateStyleSet doesn't load ua.css
> + * a few styles are common to html and SVG though

Append ", so we copy the rules below from that file."
Attachment #8496580 - Flags: review?(jwatt) → review+
I was essentially going for what Daniel said. Feel free to expand on my suggested change to more fully address that.
[Tracking Requested - why for this release]:
Requesting tracking for Firefox 33 -- this was a regression that shipped in Firefox 32 that can break interaction in SVG documents / SVG-based webapps (by not making it clear that links are clickable).  Looks like the fix should be easily & safely backportable, and it'd be nice to have this fixed in 33, so that it's only broken for one release.
A low risk fix for this will have to land today.
Flags: needinfo?(longsonr)
Flags: needinfo?(dholbert)
Flags: in-testsuite+
Approval Request Comment
[Feature/regressing bug #]: bug 1013936
[User impact if declined]: link cursor incorrect
[Describe test coverage new/current, TBPL]: patch includes mochitest
[Risks and why]: very low, just restores the old link link handling css
[String/UUID change made/needed]: none
Attachment #8496722 - Flags: approval-mozilla-beta?
Attachment #8496722 - Flags: approval-mozilla-aurora?
Comment on attachment 8496722 [details] [diff] [review]
as landed on inbound

Approving even it is not in m-c to make sure it is in beta 8
Attachment #8496722 - Flags: approval-mozilla-beta?
Attachment #8496722 - Flags: approval-mozilla-beta+
Attachment #8496722 - Flags: approval-mozilla-aurora?
Attachment #8496722 - Flags: approval-mozilla-aurora+
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Flags: needinfo?(dholbert)
Depends on: 1075479
No longer depends on: 1075479
You need to log in before you can comment on or make changes to this bug.