Skip to content

Conversation

roverwolf
Copy link
Contributor

Q A
Bug fix? yes
New feature? no
Docs? no
Issues Fix #2958
License MIT

Change the xmlns attribute to be http://www.w3.org/2000/svg for icons imported using console ux:icons:import so that the icons will correctly render in browsers and can be viewed by other image tools.

@carsonbot carsonbot added Bug Bug Fix Status: Needs Review Needs to be reviewed labels Jul 29, 2025
@smnandre
Copy link
Member

I did not know that... w3c/svgwg#738 👍

@roverwolf Could you also change the UX toolkit affected tests?

@roverwolf roverwolf requested a review from Kocal as a code owner July 30, 2025 00:21
@roverwolf
Copy link
Contributor Author

@smnandre Toolkit snapshots have been updated in the PR. Sorry, I had only checked the tests in the Icons component.

@smnandre
Copy link
Member

PHP 8.4 failures unrelated (autocomplete)

@smnandre
Copy link
Member

Thank you @roverwolf!

@smnandre smnandre added Status: Reviewed Has been reviewed by a maintainer and removed Status: Needs Review Needs to be reviewed labels Jul 30, 2025
@Kocal
Copy link
Member

Kocal commented Jul 30, 2025

Interesting, I didn't know this was so subtile

@Kocal Kocal added the Icons label Jul 30, 2025
@Kocal Kocal changed the title [ux-icons] Fix value of xmlns attribute for imported svg icons [Icons] Change xmlns attribute of downloaded icons to http://www.w3.org/2000/svg Jul 30, 2025
@Kocal Kocal force-pushed the feat/fix_xmlns_attribute branch from f9c9da3 to bdf6270 Compare July 30, 2025 06:14
@Kocal
Copy link
Member

Kocal commented Jul 30, 2025

Thank you @roverwolf.

@Kocal Kocal merged commit 43fd7e9 into symfony:2.x Jul 30, 2025
23 of 25 checks passed
@roverwolf roverwolf deleted the feat/fix_xmlns_attribute branch July 30, 2025 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Bug Fix Icons Status: Reviewed Has been reviewed by a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ux-icons] Imported icons have wrong xmlns value
4 participants