Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

@react-md/material-icons incomplete #1021

Open
Gaibhne opened this issue Dec 8, 2020 · 3 comments
Open

@react-md/material-icons incomplete #1021

Gaibhne opened this issue Dec 8, 2020 · 3 comments

Comments

@Gaibhne
Copy link

Gaibhne commented Dec 8, 2020

Describe the bug The list of classes generated for the @react-md/material-icons package is incomplete and is missing various icons.

To Reproduce Steps to reproduce the behavior:

  1. npm install --save @react-md/material-icons @react-md/icon
  2. Attempt to import for example MenuOpenSVGIcon (MD icon 'e5d2')
  3. Error output: WARNING in ./src/components/Whatever.jsx 136:29-44 - "export 'MenuOpenSVGIcon' was not found in '@react-md/material-icons'

Expected behavior Since the classes are auto generated and there is no way to use icon codes like e5d2 directly, there should be classes for all icons.

@Gaibhne Gaibhne added the bug label Dec 8, 2020
@mlaursen mlaursen added this to the v2.6.0 milestone Dec 9, 2020
@mlaursen mlaursen modified the milestones: v2.6.0, v.Unknown Jan 3, 2021
@mlaursen
Copy link
Owner

This one's going to be a bit tricky to implement. I have a feature/material-icons branch that:

  • updated my icon generator script 00b178d
  • generated all the icons d17017b

Unfortunately, there are around 6800 icons now provided by the material icons library so now my computer runs out of memory if trying to typecheck or compile. Something I could try is to just make my icon component generator a public package and users can use it to generate the icons needed for their project.

@Gaibhne
Copy link
Author

Gaibhne commented Mar 11, 2021

That does sound bad. I'm not entirely clear on the advantage of the concrete icon classes over the generic classes with the item name as a parameter, so I'm not sure how necessary those classes really are. An icon generator would definitely be a welcome addition/release to get around the current issue.

@mlaursen mlaursen modified the milestones: v.Unknown, v6.0.0 Jun 11, 2022
@mlaursen
Copy link
Owner

I'll close this with the v6 release, but there's some info here about how it'll be added. The docs will also include an example script for generating icon components.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants