Skip to content

[Icons] Change xmlns attribute of downloaded icons to http://www.w3.org/2000/svg #2959

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 30, 2025

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