ConsenSysMesh / ConsenSysMesh/rimble-ui

Refactor global icon import out of library

Open
#436 0 comments 0 reactions 1 assignee Claimed by @MikeLockz View on GitHub
Dominant language
JavaScript
Stars
455
Forks
60
PR merge metrics
No merged PRs in 30d

Description

There should not be a dependence on importing all icons into rimble-ui. The current Icon component imports `import * as RimbleIcons from @rimble/icons` all icons or `import Icon from "../Icon"` . This creates a large library bundle size that does not tree-shake properly.

Components that utilize a global import that will need refactoring:
* Icon
* ToastMessage
* Button

Refactor components to only import single icons as needed:
* EthAddress
* FileInput

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.