-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
[Mobile] - MediaUpload - Remove dashicons and use icons instead #25576
Conversation
Size Change: +1 kB (0%) Total Size: 1.17 MB
ℹ️ View Unchanged
|
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.
Tested via the demo app. This is working well for me 👍
There seems to be one other location where this might come up but I'm unsure how to recreate that case. So I'll leave that up to you
Thanks for the review @chipsnyder! I updated the other icon as well to prevent future crashes. |
Gutenberg Mobile PR
-> wordpress-mobile/gutenberg-mobile#2660Description
After this PR some dashicons were left to update causing the
MediaUpload
component to break on Android. This PR changes those icons to use some from the Icons library.How has this been tested?
Screenshots
Note: The icons are slightly different. cc @iamthomasbishop
We were using for:
Image: https://developer.wordpress.org/resource/dashicons/#format-image
Video: https://developer.wordpress.org/resource/dashicons/#format-video
Update: there was also another icon that was set as default for the
DropdownMenu
component. This component is used in theHeading
block but since this block passes its own icon, it wasn't causing issues. I updated it to prevent a future crash.Types of changes
Bug fix
Checklist: