1352979 - <use xlink:href> should not respect <base> for local references

Status: RESOLVED DUPLICATE of bug 1357432

(Core :: SVG, defect)

Description

[Anthony Ricaud (:rik)]

Steps to reproduce:
Open the testcase

Actual result:
Logo is invisible

Expected result:
Logo is displayed

Works as expected: Chrome, Safari, IE 10 and 11
Works as actual: Firefox, Edge

A lot of people are running into this issue and fixing by using absolute references: https://gist.github.com/leonderijke/c5cf7c5b2e424c0061d2

I think <base> should not be used for local references aka #foo.

Comment 1

[Robert Longson [:longsonr]]

Please approach w3c first if you want things to change and get agreement on a specification update. The SVG specification currently says to generate an absolute URL

https://www.w3.org/TR/SVG2/linking.html#processingURL-absolute says 

In contrast, a base element affects relative URLs in any SVG or HTML document, by altering the document base URL. 

so you'll need to get the SVG specification changed first.

Comment 2

[Karl Dubost💡 :karlcow]

Thanks Anthony for the report. 
The closest I find so far is 
* Document.baseURI getter doesn't return the parent document's base URL for iframes 
  https://bugs.chromium.org/p/chromium/issues/detail?id=484930

I opened another issue on the Chromium project to see what would be the comments.
https://bugs.chromium.org/p/chromium/issues/detail?id=707758

I checked in Blink (Opera), the use.baseURI is properly set on the element but is not used for deferencing the xlink:href

Comment 3

[Karl Dubost💡 :karlcow]

Attachment: test-xlink-href-svg-base.html

Test case showing the issue.

Comment 4

[Anthony Ricaud (:rik)]

Robert: What's the best forum to bring up those issues?

Comment 5

[Robert Longson [:longsonr]]

https://lists.w3.org/Archives/Public/www-svg/

Comment 6

[Anthony Ricaud (:rik)]

Actually, that issue has already been brought up:

"As defined in CSS Values and Units, a fragment-only URL in a style property must be treated as a same-document URL reference, regardless of the file in which the property was declared."

https://github.com/w3c/svgwg/issues/61
https://github.com/w3c/svgwg/commit/f378b15b95841cb1297eda7b1a6a2ca2d549ee71

Reopening per spec.

Comment 7

[Karl Dubost💡 :karlcow]

Yes and in addition, the issue was closed as WONTFIX on Chrome project because of Web Compatibility issues.

https://bugs.chromium.org/p/chromium/issues/detail?id=707758#c3
> This was changed via issue 470608 after we had attempted to implement the Gecko behavior in issue 449027 and got push-back (possibly because this "broken" behavior had existed in WebKit for quite a while.)


Given that Chrome, Safari, IE 10 and 11 works the same, I would be in favor of following the behavior of others.

Comment 8

[Anthony Ricaud (:rik)]

It has been fixed in Edge.

Comment 9

[Anthony Ricaud (:rik)]

Robert : Do you think this bug is easy enough to fix that it could be a mentored bug?

Comment 10

[Robert Longson [:longsonr]]

Seems to be the same issue as bug 1357432 which already has a patch. Maybe just dup to that one.