Skip to content

refactor(web): drop unused icons#2215

Merged
imobachgs merged 2 commits intomasterfrom
drop-unused-icons
Mar 27, 2025
Merged

refactor(web): drop unused icons#2215
imobachgs merged 2 commits intomasterfrom
drop-unused-icons

Conversation

@dgdavid
Copy link
Contributor

@dgdavid dgdavid commented Mar 27, 2025

As the project has evolved, many icons are no longer used, but they were still being imported and included in the final bundle.

This PR removes the unused icons and also drops a dependency that is no longer needed after the clean-up.

Since there are no visual changes for users, no entry has been added to the changes file.

dgdavid added 2 commits March 27, 2025 08:50
Because we stop using the system icons while ago.

@icons-pack/react-simple-icons
@dgdavid dgdavid requested a review from imobachgs March 27, 2025 08:58
Copy link
Contributor

@joseivanlopez joseivanlopez left a comment

Choose a reason for hiding this comment

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

LGTM

@imobachgs imobachgs merged commit fae22a8 into master Mar 27, 2025
7 checks passed
@imobachgs imobachgs deleted the drop-unused-icons branch March 27, 2025 11:03
@imobachgs imobachgs mentioned this pull request Mar 27, 2025
imobachgs added a commit that referenced this pull request Mar 27, 2025
@imobachgs imobachgs mentioned this pull request Apr 22, 2025
imobachgs added a commit that referenced this pull request Apr 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants