Closed Bug 1130888 Opened 5 years ago Closed 5 years ago

SVG text on path fill color does not render properly when preceded by RTL text

Categories

(Core :: SVG, defect)

35 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla38
Tracking Status
firefox35 --- wontfix
firefox36 --- wontfix
firefox37 --- fixed
firefox38 --- fixed
firefox-esr31 --- unaffected
b2g-v2.2 --- fixed
b2g-master --- fixed

People

(Reporter: bridgeproductions, Assigned: longsonr)

References

Details

(Keywords: regression)

Attachments

(2 files)

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

Steps to reproduce:

Since updating Firefox to version 35 from the previous version 34 some features in a website I developed are not working as before.
When updating to the newest 36 version this problem still occurs, and of course I double checked and reverted back to 34 to confirm.


Actual results:

The problem itself is that SVG text on path fill color doesn’t respond and for some reason renders as white.

I don’t know if it helps that I’ll add more details, but anyway in my case the SVG text on path is also rotating when mouse over (CSS3 transitions and transforms).
By the way, I chose to rotate text and not code a SMIL animation (SVG text along an arbitrary path animation) as it doesn’t work with Firefox (http://css-tricks.com/guide-svg-animations-smil/).


Expected results:

The website that I’m referring to is: http://www.gaaal.com/ 
The SVG tags are located in: html/body/main/div/a
In each a tag there are two SVG tags that are in position absolute to the containing div.
Of course when checking the website with other browsers fill color rendering is working as it supposed to.
Hardware: x86_64 → Other
Summary: SVG fill color does not render properly (SVG+CSS3 Transitions) → SVG text on path fill color does not render properly (SVG+CSS3 Transitions)
A couple of things would help a lot. A reduced testcase I.e. the smallest code necessary to reproduce the problem rather than your whole website ideally attached to this bug using the Add an attachement link above.

Secondly using moz-regression http://mozilla.github.io/mozregression/ to find out exactly when things changed.

Do you think you could do either of those things?
Flags: needinfo?(bridgeproductions)
Removing the span makes the textPath change colour.
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=cdbdea7d3705&tochange=f2003d572d53

Triggered by:
f2003d572d53	Robert Longson — Bug 1078031 - get rid of renderMode. r=jwatt
Status: UNCONFIRMED → NEW
Ever confirmed: true
Component: Untriaged → SVG
Product: Firefox → Core
Flags: needinfo?(bridgeproductions)
OS: Windows 7 → All
Hardware: Other → All
Summary: SVG text on path fill color does not render properly (SVG+CSS3 Transitions) → SVG text on path fill color does not render properly when preceded by RTL text
Attached patch text.txtSplinter Review
Assignee: nobody → longsonr
Attachment #8561707 - Flags: review?(cam)
Comment on attachment 8561707 [details] [diff] [review]
text.txt

Review of attachment 8561707 [details] [diff] [review]:
-----------------------------------------------------------------

::: layout/reftests/svg/textPath-05.html
@@ +4,5 @@
> +    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
> +</head>
> +<body>
> +    <span style="display:none;"> דניאל גל</span>
> +    <svg width="100%" height="100%"> 

Trailing space.
Attachment #8561707 - Flags: review?(cam) → review+
https://hg.mozilla.org/mozilla-central/rev/cbb846c9b107
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Comment on attachment 8561707 [details] [diff] [review]
text.txt

Approval Request Comment
[Feature/regressing bug #]: bug 1078031
[User impact if declined]: Some SVG text displayed in the wrong colour
[Describe test coverage new/current, TreeHerder]: includes a new reftest
[Risks and why]: low (change is small and similar to code in nsTextFrame)
[String/UUID change made/needed]: none
Attachment #8561707 - Flags: approval-mozilla-beta?
Attachment #8561707 - Flags: approval-mozilla-aurora?
Comment on attachment 8561707 [details] [diff] [review]
text.txt

We already shipped 35 with this bug. This is really late in the 36 cycle and I saw enough regressions in the SVG code to not accept last minute changes in the code.
Attachment #8561707 - Flags: approval-mozilla-beta?
Attachment #8561707 - Flags: approval-mozilla-beta-
Attachment #8561707 - Flags: approval-mozilla-aurora?
Attachment #8561707 - Flags: approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.