Icon Doc page inconsistencies
See original GitHub issueEdit: Looks like a couple of the icon issues have been reported here too: https://github.com/Semantic-Org/Semantic-UI/issues/6175. That being said, there are still a lot of other issues I mention below
Love the new addition of Font Awesome 5 to SUI! Very nice work! However, I was looking through the categories and noticed several inconsistencies between the Semantic UI Icons doc page and the documentation on Font Awesome’s website.
- The
accessible icon
(the modern wheelchair icon) is missing from Accessibility, Health, Users and People, and Vehicles categories (it’s in the ‘Brands’ section only on the SUI doc page, it would be nice to include it in these other categories) - The youtube icon is missing from the Audio & Video section
- The Chess Rook icon is misspelled in SUI, it has
chess rock
, it should bechess rook
- The ‘Currency’ category icons have several inconsistencies, Bitcoin and others aren’t listed here
- SUI has the
user doctor
icon instead ofuser md
like in Font Awesome, maybe make an alias so users could use either - The ‘Payments’ category icons also have several inconsistencies, amazon and apply pay are missing as well as many others
I also found several other icons that appear to be inaccurate on the docs page. I’ve attached images of them.
Thanks so much and let me know if I can help in any way or if you want me to submit a PR for these. Are the main SUI documentation pages something we can submit PR’s for?
Best, -Justin
These are the incorrect icons:
These are what they should look like based on the FA 5 docs:
Issue Analytics
- State:
- Created 6 years ago
- Reactions:1
- Comments:6 (4 by maintainers)
😄 No worries, I bet! You did a great job though! Kudos!
Yeah, it looks like you fixed the arrows alternate horizontal, arrows alternate vertical, and the exchange alternate. It looks like the code, code branch, file code, and file code outline aren’t mentioned in that new PR https://github.com/Semantic-Org/Semantic-UI/pull/6181 though.
Let me know if you have any questions on the others and thanks for the swift reply! You’re crushing it @hammy2899 👍
This will be out before Friday with @hammy2899’s work