Skip to content
This repository was archived by the owner on Sep 30, 2025. It is now read-only.

Conversation

sainihas
Copy link
Contributor

@sainihas sainihas commented Nov 6, 2023

WHY are these changes introduced?

The current metaobject icons are not aligned with the uplifted icons in Figma
This PR updates them to align it with FIgma.

From Figma From prod
Screenshot 2023-11-06 at 4 45 41 PM Screenshot 2023-11-06 at 4 45 57 PM

🎩 checklist

@sainihas sainihas force-pushed the update-metaobject-icon branch 3 times, most recently from dc9f054 to 8f6db0d Compare November 6, 2023 22:02
@sainihas sainihas requested a review from heyjoethomas November 6, 2023 22:03
@mkoenig-shopify mkoenig-shopify self-requested a review November 6, 2023 22:07
Copy link

@mkoenig-shopify mkoenig-shopify left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so much better - thank you Sai!

Copy link

@danloomer danloomer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beauty 🥇

@heyjoethomas heyjoethomas linked an issue Nov 6, 2023 that may be closed by this pull request
@sainihas sainihas force-pushed the update-metaobject-icon branch 2 times, most recently from 3d1fb48 to cca215c Compare November 6, 2023 22:44
@sainihas sainihas force-pushed the update-metaobject-icon branch from cca215c to 2821225 Compare November 6, 2023 22:47
@sainihas sainihas merged commit e55ccba into main Nov 6, 2023
@sainihas sainihas deleted the update-metaobject-icon branch November 6, 2023 23:00
alex-page added a commit that referenced this pull request Nov 8, 2023
alex-page added a commit that referenced this pull request Nov 8, 2023
Bringing the changes from #11119
back into the `main` branch with the old names. Thanks @sainihas for the
contribution!

Co-authored-by: Sai Nihas <41344212+sainihas@users.noreply.github.com>
AnnaCheba pushed a commit to AnnaCheba/polaris that referenced this pull request Apr 22, 2024
…ify#11119)

<!--
  ☝️How to write a good PR title:
- Prefix it with [ComponentName] (if applicable), for example: [Button]
  - Start with a verb, for example: Add, Delete, Improve, Fix…
  - Give as much context as necessary and as little as possible
  - Prefix it with [WIP] while it’s a work in progress
-->

### WHY are these changes introduced?

The current metaobject icons are not aligned with the [uplifted icons in
Figma](https://www.figma.com/file/fVIazfJNe3AOJTJmBKgTO9/Polaris-Gen-3-Icons?type=design&node-id=11505%3A221&mode=dev)
This PR updates them to align it with FIgma.

| From Figma  | From prod |
| ------------- | ------------- |
| <img width="393" alt="Screenshot 2023-11-06 at 4 45 41 PM"
src="https://github.com/Shopify/polaris/assets/41344212/454b5013-2b16-49bc-a801-547643bc8894">
| <img width="209" alt="Screenshot 2023-11-06 at 4 45 57 PM"
src="https://github.com/Shopify/polaris/assets/41344212/eb2eb211-e700-4e32-9348-7ff6ffcc09d5">
|



### 🎩 checklist

- [x] Tested on
[mobile](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md#cross-browser-testing)
- [x] Tested on [multiple
browsers](https://help.shopify.com/en/manual/shopify-admin/supported-browsers)
- [x] Tested for
[accessibility](https://github.com/Shopify/polaris/blob/main/documentation/Accessibility%20testing.md)
- [x] Updated the component's `README.md` with documentation changes
- [x] [Tophatted
documentation](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting%20documentation.md)
changes in the style guide
AnnaCheba pushed a commit to AnnaCheba/polaris that referenced this pull request Apr 22, 2024
ascherkus pushed a commit to ascherkus/polaris that referenced this pull request Feb 19, 2025
…ify#11136)

Bringing the changes from Shopify#11119
back into the `main` branch with the old names. Thanks @sainihas for the
contribution!

Co-authored-by: Sai Nihas <41344212+sainihas@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Icon]: Update icons <Metaobject> and <MetaobjectReference>
5 participants