-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Icons: Update cloud upload and add cloud download icon. #65906
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
Conversation
|
👋 Thanks for your first Pull Request and for helping build the future of Gutenberg and WordPress, @matt-west! In case you missed it, we'd love to have you join us in our Slack community. If you want to learn more about WordPress development in general, check out the Core Handbook full of helpful information. |
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
|
@jasmussen @jameskoster Would you mind taking a look at this proposal for updating the This work came out of a new push/pull feature we’re building for Studio, but thought it would be useful to have in Core too. I’ve maintained the current cloud dimensions which fit snuggly within the |
|
At a glance, looks substantially improved, solid work. Do note that, at least for the moment and this is pending better ideas, the "cloud upload" button is the icon-fallback for "Save draft", when the post editor is on mobile. This was never great, and worth looking at separate mobile editing improvements, so that it can perhaps become a text button on mobile instead. But does this change anything as far as this work? My instinct, no, it's fine. |
Agreed. At the resolution shown below, it could make more sense to use an icon for the Preview button and display the Save draft text instead. There may not be enough space to accommodate that in non-English languages though. |
|
Looks good to me, nice work. But lets update the svgs so they include a single path. We should add a changelog entry too. |
|
Thanks @jameskoster. I’ve reduced the icons to a single path and added entries to the changelog. |
jameskoster
left a comment
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.
Looks good to me. Thanks so much :)
Head branch was pushed to by a user without write access
a501bde to
612a983
Compare
|
@jameskoster Can you please set this to merge again. I rebased to bring in some changes from trunk that fixed a broken test but it disabled the auto-merge. |
|
Done! Thanks again. |

What?
Update the cloud upload icon to make the arrow more readable. This PR also adds a counterpart for cloud download this is not currently available.
Figma assets
Why?
The current cloud upload icon likes whitespace around the arrow, which makes it difficult to quickly read at smaller sizes.
There is no cloud download icon equivalent in the current icon package, however this could be useful for interfaces that include push and pull actions.
Testing Instructions
Open Storybook and observe the icons.
Screenshots or screencast
The cloud upload icon is currently used in the editor toolbar on mobile.
Here’s a comparison of how that toolbar looks with the current and proposed icons.
Here’s a continuity test showing the new icons alongside others in the icon package.