-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update metafield icon #11680
Update metafield icon #11680
Conversation
Ran optimization and added changeset
Previous svg had uneven corners causing the icon to feel unbalanced.
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.
Hey @Rusty-UX . Thanks for throwing up the PR. I fixed the following to get all checks to pass and the final svg's to be 👍
- Added a changeset (required or build will fail). You can do this next by simply running
yarn changeset
before submitting the PR. - Optimized the icons (this is a new required step). Do this by running
Yarn build && yarn optimize
insidepolaris-icons
. - The border radius on the top corners were not even so the icon wasn't symmetrical. Just evened it all out so it all looks 🤌 .
Thanks again for the contribution 🤝
@Rusty-UX make sure your changeset file is related to what this code does. In this case it should say "Update metafield icon design". |
This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## @shopify/polaris-icons@8.8.0 ### Minor Changes - [#11680](#11680) [`423ffb755`](423ffb7) Thanks [@Rusty-UX](https://github.com/Rusty-UX)! - Update metafield icon design ## @shopify/polaris@12.21.0 ### Minor Changes - [#11650](#11650) [`00e276407`](00e2764) Thanks [@kyledurand](https://github.com/kyledurand)! - Added `cover` as an `preferredPosition` to `Popover` - [#11696](#11696) [`6a1a5a6c4`](6a1a5a6) Thanks [@mmapplebeck](https://github.com/mmapplebeck)! - [Frame] Fixed regression that caused sidebar to render over Admin app content - [#11708](#11708) [`90a3cb1fb`](90a3cb1) Thanks [@sam-b-rose](https://github.com/sam-b-rose)! - Increase icon size using viewBox for mobile screen sizes. This will enlarge icons on mobile without affecting the icon wrapper. ### Patch Changes - [#11689](#11689) [`da680f36a`](da680f3) Thanks [@kyledurand](https://github.com/kyledurand)! - Fixed touch device styling for removable large Tags - [#11693](#11693) [`3554a1004`](3554a10) Thanks [@mrcthms](https://github.com/mrcthms)! - Updated BulkActionButton to render the Tooltip below the button, not above it - Updated dependencies \[[`423ffb755`](423ffb7)]: - @shopify/polaris-icons@8.8.0 ## polaris.shopify.com@0.65.0 ### Minor Changes - [#11703](#11703) [`add5ffbea`](add5ffb) Thanks [@lgriffee](https://github.com/lgriffee)! - Added shadow token design guidance ### Patch Changes - Updated dependencies \[[`423ffb755`](423ffb7), [`da680f36a`](da680f3), [`00e276407`](00e2764), [`3554a1004`](3554a10), [`6a1a5a6c4`](6a1a5a6), [`90a3cb1fb`](90a3cb1)]: - @shopify/polaris-icons@8.8.0 - @shopify/polaris@12.21.0 Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
# Updating metafield icon design ### WHY are these changes introduced? Design updated to be have a bit more organic shapes, based on feedback ### Before / After @2x <img width="657" alt="render-test" src="https://github.com/Shopify/polaris/assets/4304867/741aa66d-4348-42e3-a9cf-3ecc99c44a87"> @1x ![render-1x](https://github.com/Shopify/polaris/assets/4304867/5681f8b5-5bff-42d8-a16a-01dc20c69b6b) --------- Co-authored-by: Joe Thomas <joe.thomas@shopify.com>
Updating metafield icon design
WHY are these changes introduced?
Design updated to be have a bit more organic shapes, based on feedback
Before / After
@2x
![render-test](https://private-user-images.githubusercontent.com/4304867/309363488-741aa66d-4348-42e3-a9cf-3ecc99c44a87.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE3MDk4MDYsIm5iZiI6MTcyMTcwOTUwNiwicGF0aCI6Ii80MzA0ODY3LzMwOTM2MzQ4OC03NDFhYTY2ZC00MzQ4LTQyZTMtYTljZi0zZWNjOTljNDRhODcucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcyMyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MjNUMDQzODI2WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9ZTk2Njg5MzY0ODc3MGM5NDIzN2E1ODJlZDE0OGFiZjU3NjEyMmZkZjNjN2YxNTgxZDFiMWU3NzM3YTNlMGI1ZSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ._HSqe3DRi9E0EnQZn014TpXv6aE4jHBdsRwUohu0uRc)
@1x
![render-1x](https://private-user-images.githubusercontent.com/4304867/309363484-5681f8b5-5bff-42d8-a16a-01dc20c69b6b.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE3MDk4MDYsIm5iZiI6MTcyMTcwOTUwNiwicGF0aCI6Ii80MzA0ODY3LzMwOTM2MzQ4NC01NjgxZjhiNS01YmZmLTQyZDgtYTE2YS0wMWRjMjBjNjliNmIucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcyMyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MjNUMDQzODI2WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9Yjc4NTFkOTkwMTlkOTI2YTllMjlmNDdmY2M5ZDNmNDIxNDRlOTBjMzZmMWU0ZTYzOTM2MjNjOWQ0YmRiZjIxZCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.0XSgvOyvzjuh5jxnBK8jlTvhUevW_-j7OIa_aaol_n8)