-
Notifications
You must be signed in to change notification settings - Fork 59
backward compatible icons and using lucide icons #963
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
backward compatible icons and using lucide icons #963
Conversation
…and-backwards-compatible # Conflicts: # src/packages/core/property-editor/uis/tiny-mce/config/manifests.ts # src/packages/core/property-editor/uis/tiny-mce/manifests.ts
…://github.com/umbraco/Umbraco.CMS.Backoffice into feature/lucide-icons-and-backwards-compatible
…m Props, so we can theme them and change as time goes.
Now you are touching all the icons anyway, can you then move the |
…and-backwards-compatible
Azure Static Web Apps: Your stage site is ready! Visit it here: https://ambitious-stone-0033b3603-963.westeurope.1.azurestaticapps.net |
Azure Static Web Apps: Your stage site is ready! Visit it here: https://ambitious-stone-0033b3603-963.westeurope.1.azurestaticapps.net |
2 similar comments
Azure Static Web Apps: Your stage site is ready! Visit it here: https://ambitious-stone-0033b3603-963.westeurope.1.azurestaticapps.net |
Azure Static Web Apps: Your stage site is ready! Visit it here: https://ambitious-stone-0033b3603-963.westeurope.1.azurestaticapps.net |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I replaced a few more icons (I think we got them all now) and fixed a small issue with the icon picker. LGTM.
Make icon names backward compatible.
And appends new icons from Lucide, which replaces existing icons.
Add legacy option for icons, so they cannot be picked in the Icon Picker.
Requires two PRs of UI Library to be handled first and upgraded to a version of UUI where that takes part:
umbraco/Umbraco.UI#629
umbraco/Umbraco.UI#649
Description
Types of changes
Motivation and context
How to test?
Screenshots (if appropriate)
Checklist