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
[UI]update rocket icon to emoji #18729
Conversation
update rocket icon to emoji
add rocket icon image
This reverts commit 1241174.
update icon screenshot
If the font icon isn't in use anymore, I guess we can also remove it again |
remove rocket icon font
This reverts commit 7a8e13c.
@sgiehl it seems like there are lots of places using Rocket Icon. Do we want to replace all of them with emojis? Otherwise, I think removing the icon will break some parts, some js Tests has click event on |
@tsteur Shall we replace all usages of the rocket icon with the emoji, or only that specific one? |
not needed for now @sgiehl |
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.
merged in the last changes from 4.x-dev. If related tests are passing, this should be good to merge.
Description:
Fixes: #18700
update rocket icon to emoji.
It seems like the emoji Character does not exist in the Travis, Exist in GitHub action. see here
Review