The foundation-sites dependency still exists and should be removed. There is nothing that currently uses it. The Icon component uses the class names for its icons fi- but doesn't directly depend on it.
The task should be as follows:
npm uninstall foundation-sites
in src/Icon.js, make it so that fi- is no longer prepended to iconName. This means that users will now have to use the full class name when using the component
bryanjos
commented
4 years ago
I'm submitting a ...
The
foundation-sitesdependency still exists and should be removed. There is nothing that currently uses it. The Icon component uses the class names for its iconsfi-but doesn't directly depend on it.The task should be as follows:
npm uninstall foundation-sitessrc/Icon.js, make it so thatfi-is no longer prepended toiconName. This means that users will now have to use the full class name when using the componentdocs-src/src/examples/Icon/Foundation.js.example