Decathlon / vitamin-web

Decathlon Design System UI components for web applications
https://decathlon.github.io/vitamin-web
Apache License 2.0
282 stars 76 forks source link

feat: new icons #1323

Closed lauthieb closed 1 year ago

lauthieb commented 1 year ago

Changes description

Close #1307

Context

Checklist

Does this introduce a breaking change?

lauthieb commented 1 year ago

Hi! @thibault-mahe I let you check this 🙏

thibault-mahe commented 1 year ago

The problem has been identified and a global solution (including existing icons) should be proposed in a future PR. I merge as is. Thank you!

lauthieb commented 1 year ago

Thanks @thibault-mahe, as discussed, this is another problem as we already have this problem for a lot of icons already in main branch. I think we can merge this and follow the problem with #1315 for icons & assets