-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Evaluate 16, 20 and 24 icons for redundancy #5597
Comments
Hey there! Going to close this out due to inactivity, feel free to comment and I can re-open! |
@conradennis could also evaluate these. otherwise I'll try to chip away at these! |
Reopening to finish this. |
duplicate icons with different names in the library:
updated icons to reduce need for redundancy:
there are also a few icons that can be reworked to render better at smaller sizes but when proper tweaks are made they just look worse/unbalanced compared to their original versions. I'll need to work with these a bit more:
any icons I update will be included in my next batch PR later this week so once I make that issue I'll reference it here and then when it's merged we should be able to close this one. |
Thank you, Conrad! 🙌🏻 |
Submitting new and updated UI icons from September and October all updates and additions approved by BXD 193 net new icons 48 updated icons updated categories.yml with additions updated ui-icon-master.ai with additions and updates Closes: carbon-design-system#5597
Submitting new and updated UI icons from September and October all updates and additions approved by BXD 193 net new icons 48 updated icons updated categories.yml with additions updated ui-icon-master.ai with additions and updates Closes: carbon-design-system#5597
PR including all updated icons mentioned above: #7126 we should be able to remove all of the icons Connors listed and close the issue |
* feat(UI icons): new icons september and october Submitting new and updated UI icons from September and October all updates and additions approved by BXD 193 net new icons 48 updated icons updated categories.yml with additions updated ui-icon-master.ai with additions and updates Closes: #5597 * fix(icons): update metadata * chore(icons): update category metadata * test(e2e): update public api snapshots * test(react): update snapshots * test(icons-vue): update e2e snapshots for vue Co-authored-by: Lauren Rice <[email protected]> Co-authored-by: D.A. Kahn <[email protected]> Co-authored-by: DAK <[email protected]> Co-authored-by: Josh Black <[email protected]> Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
* feat(UI icons): new icons september october Submitting new and updated UI icons from September and October all updates and additions approved by BXD 193 net new icons 48 updated icons updated categories.yml with additions updated ui-icon-master.ai with additions and updates Closes: #5597 * feat(UI icons): new icons july august 2021 Submitting new and updated UI icons from July and August 2021 all updates and additions approved by BXD 89 net new icons 3 updated icons updated categories.yml with additions updated ui-icon-master.ai with additions and updates * chore(icons): update metadata and snapshots Co-authored-by: Josh Black <[email protected]> Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Last batch of icons to evaluate for elimination!:
The text was updated successfully, but these errors were encountered: