-
Notifications
You must be signed in to change notification settings - Fork 5
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
Icon purge? #205
Comments
Unused icons (from github search)
|
I Agree with this, just probably need people to go through their verticals and make sure we don't need any of these.
I defend that stuff should only be in uStyle if it's shared, so if it's only used within mobiles, I'm very happy to cater for them myself |
@ricardomccerqueira Can you clarify that those are internal icons you are using are you are indeed using the ustyle icons? |
mobiles-web is using a separate version of those Icons: Feel free to delete them from uStyle if you want. |
@annez We don't use any of the icons that you mentioned in your second comment. We do use |
We currently have quite a few unused icons in our repository. Would be good to purge ones we do not use.
Main reasons:
If we come up with a better solution on doing the icons, then we should go with that.
The text was updated successfully, but these errors were encountered: