Skip to content

Use SVGR for icons#6808

Merged
valadas merged 2 commits into
dnnsoftware:developfrom
bdukes:svgr-for-icons
Nov 24, 2025
Merged

Use SVGR for icons#6808
valadas merged 2 commits into
dnnsoftware:developfrom
bdukes:svgr-for-icons

Conversation

@bdukes

@bdukes bdukes commented Nov 20, 2025

Copy link
Copy Markdown
Contributor

Summary

This PR updates the approach to loading SVG icons in the Persona Bar to use SVGR's Webpack loader to convert the SVG files into components (while also optimizing them).

@bdukes bdukes added this to the 10.1.3 milestone Nov 20, 2025
@bdukes
bdukes force-pushed the svgr-for-icons branch 3 times, most recently from 4b947d3 to df963ef Compare November 20, 2025 23:27

@valadas valadas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is awesome, now I downloaded the CI build and went to every module, everything worked fine except the Users module blows up...

@bdukes

bdukes commented Nov 21, 2025

Copy link
Copy Markdown
Contributor Author

Thanks for testing @valadas, I've fixed the issue in the Users module

@bdukes
bdukes requested a review from valadas November 21, 2025 14:37

@donker donker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@mitchelsellers mitchelsellers left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!

@valadas valadas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent!

@valadas
valadas merged commit c0b23cd into dnnsoftware:develop Nov 24, 2025
3 checks passed
@valadas valadas modified the milestones: 10.1.3, 10.2.0 Dec 3, 2025
@david-poindexter

Copy link
Copy Markdown
Contributor

I'm late to the party, but just wanted to say thanks @bdukes

@bdukes
bdukes deleted the svgr-for-icons branch December 17, 2025 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants