New issue
Advanced search Search tips

Issue 680887 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Jan 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug



Sign in to add a comment

xlink and filter stop work with base element in head

Reported by jiangyue...@gmail.com, Jan 13 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2785.101 Safari/537.36

Steps to reproduce the problem:
1. place the test.html in a directory and start server with port 8000
2. visit http://127.0.0.1:8000/test.html
3. 

What is the expected behavior?
the svg and canvas behave normally

What went wrong?
When base href set.
1. The first text element can not display filter.
2. The second text element with a full path filter url works.
3. When convert svg to canvas with XMLSerializer. The problem 1 and 2 just reverse. Full path filter stop working and the other is good. xlink behaves just like filter.

4. When page url contains parentheses, the full path filter stop working too. And no solution. 
xlink behave well under this situation.

Did this work before? N/A 

Does this work in other browsers? N/A

Chrome version: 53.0.2785.101  Channel: n/a
OS Version: 6.1 (Windows 7, Windows Server 2008 R2)
Flash Version: Shockwave Flash 24.0 r0

Problem 1,2,3 works well on latest Opera with Blink engine.
 
test.html
2.9 KB View Download

Comment 1 by f...@opera.com, Jan 13 2017

I think 1, 2 and 3 should've been fixed by  issue 470608  and work since Chromium 54 (Opera 41.)

So I guess that leaves 4, and isn't that just a case where you need to add "" around the string (i.e setAttribute('filter','url("'+window.location.href+'#solid")')) since in the case of this will get passed through the CSS parser, which might tokenize this the wrong way (i.e stop at the first ')', not the second.)
Thanks for response.
I have checked the solution, it works. My fault.
Then this issue can be closed now.
Thanks.

Comment 3 by f...@opera.com, Jan 13 2017

Status: WontFix (was: Unconfirmed)

Sign in to add a comment