update nuxt icons #2119
update nuxt icons #2119
Conversation
PKief
force-pushed
the
main
branch
3 times, most recently
from
640fb8a to
d5912bb
Compare
|
Hey @PKief it's been a while since I opened this PR, and I'd like to cleanup my dashboard. Can you please tell me if this is a wanted addition? And in that case, if this PR is ok or if it needs some edits? (Same thing for some other PR's that I made in the past, that maybe need to be updated: #2118, #1016, #1011) Thank you :) |
PreviewThank you for creating a pull request. This preview shows you how your changes will look on the different themes: You can find more information how to contribute in the contribution guidelines. |
There was a problem hiding this comment.
This one looks ok even if the contrast is not so good as in the previous icons. But let's go with the flow. You can close the other ones #2118, #1016, #1011, I don't like them, they are very specific and I guess they are not commonly used.\
The criticism has nothing to do with your work, sometimes it just doesn't fit. I really appreciate the effort you have put into it. Thank you for that.
Merge SuccessfulThanks for your contribution! 🎉 The changes will be part of the upcoming update on the marketplace. |
|
Thanks for this, if you want to tweak the colors I can try to see if I can find a better combination. About the other PRs, no problems at all, the dude abides ;-) Like I said I only wanted to declutter a bit my dashboard and I have no problems in closing them. |
Update nuxt icons according to the new branding
https://nuxt.com/design-kit