Skip to content

Remove FontAwesome and Hero icons from plenum and bundle#108

Merged
DerekAgility merged 1 commit intomainfrom
dlo/remove-fa-hero-icons
Mar 17, 2026
Merged

Remove FontAwesome and Hero icons from plenum and bundle#108
DerekAgility merged 1 commit intomainfrom
dlo/remove-fa-hero-icons

Conversation

@DerekAgility
Copy link
Copy Markdown
Contributor

@DerekAgility DerekAgility commented Mar 16, 2026

This is the first part of reducing the main manager-app bundle size.

In plenum, we are removing the packages for Font Awesome icons (the only usage of it was in one of the stories) and Hero Icon (not even used). This will decrease the bundled package for plenum.

To decrease it just a little bit more, I have removed the import for the entire package of TablerIcons and only import those that we need.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
plenum-ui Ready Ready Preview, Comment Mar 16, 2026 5:57pm

Request Review

IconTrash,
IconTrashFilled,
IconUpload,
IconX,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Going forward, if any component requires another TablerIcon, we will need to import it here

IconTrash,
IconTrashFilled,
IconUpload,
IconX,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

And then add to the epxort list here

@DerekAgility DerekAgility requested a review from jules-exel March 16, 2026 19:15
@DerekAgility DerekAgility merged commit ec2a277 into main Mar 17, 2026
4 checks passed
@DerekAgility DerekAgility deleted the dlo/remove-fa-hero-icons branch March 17, 2026 13:36
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.

2 participants