-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Missing icons #6464
Missing icons #6464
Comments
Are you referrring to the Constants? PR is welcome. Updated to PrimeIcons 7.0.0 but did not go through and update all the constants. |
@melloware Please, have a look at my comments at https://github.com/primefaces/primereact/pull/6465/files#r1578160473. |
OK icons fixed. Also if you are concerned any are missing I pulled them right from here: https://github.com/primefaces/primeicons/blob/master/primeicons.css |
I would like to insist on the fact that "pi-circle-off", "pi-circle-on" and "pi-file-o" are not part of the documentation. In that sense, primeicons.css seems outdated itself. Also, both these classes properly separate the word "and": .pi-arrow-up-right-and-arrow-down-left-from-center
.pi-arrow-down-left-and-arrow-up-right-to-center Which is not the case with one of their constants:
|
I am confused. Are you saying pi circle on is or isn't in primeicons.css. I pulled the CSS directly from the PrimeIcons repo? |
I'm saying those three icons aren't listed in https://primereact.org/icons. |
Looks like the JSON file from PrimeIcons was out of date: https://github.com/primefaces/primeicons/blob/master/selection.json |
Describe the bug
It seems the last batch of icons did not make it to PrimeReact... e.g. "gauge", "warehouse" and "hammer" are missing. In total, there are 311 icons, but only 261 types.
(Similar to: #5333.)
Reproducer
No response
PrimeReact version
10.6.3
React version
17.x
Language
TypeScript
Build / Runtime
Create React App (CRA)
Browser(s)
No response
Steps to reproduce the behavior
No response
Expected behavior
No response
The text was updated successfully, but these errors were encountered: