Skip to content
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

[Icons] Adds 4 new icons with new shapes #12449

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

arthurgouveia
Copy link
Contributor

@arthurgouveia arthurgouveia commented Jul 24, 2024

Closes #12010
Closes #12011
Closes #12012
Closes #12013

WHY are these changes introduced?

This adds 4 new icons to Polaris to represent shapes.

Screenshot 2024-07-24 at 11 08 21 AM

Icon Figma link
https://www.figma.com/design/x3mlbLESgBoqQrUAHz570E/Editor-improvements?node-id=436%3A35753&t=GWGMqbgUPWaGJAJk-1

WHAT is this pull request doing?

Adding icons

🎩 checklist

@alex-page
Copy link
Member

@arthurgouveia what is the timeline and necessity for these icons? We are rebuilding the icons library and if this could wait that would be appreciated. Can you use these svg files in /web?

@arthurgouveia
Copy link
Contributor Author

@alex-page feel free to deprioritze this entirely. It is a tech debt and right now we are hosting that inside Email repo. We just ideally want to fetch from Polaris directly when the time is right.

cc @mpboucher3

@alex-page
Copy link
Member

Thank you! I will keep this in mind in the future. Feel free to nudge me in a couple of weeks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants