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

Add "<<" and ">>" icons #17367

Closed
georgewrmarshall opened this issue Jan 24, 2023 · 2 comments · Fixed by #17654
Closed

Add "<<" and ">>" icons #17367

georgewrmarshall opened this issue Jan 24, 2023 · 2 comments · Fixed by #17654
Assignees
Labels
area-UI Relating to the user interface. team-design-system All issues relating to design system in Extension

Comments

@georgewrmarshall
Copy link
Contributor

georgewrmarshall commented Jan 24, 2023

Blocked by MetaMask/design-tokens#283

Description

Add arrow-double-left and arrow-double-right icons to the Icon component

Technical Description

  • Add provided svgs to the app/images/icons

Acceptance Criteria

  • There are "<<" and ">>" icons

References

@georgewrmarshall georgewrmarshall added the team-design-system All issues relating to design system in Extension label Jan 24, 2023
@georgewrmarshall georgewrmarshall added the area-UI Relating to the user interface. label Jan 24, 2023
@Akatori-Design
Copy link

@garrettbear @georgewrmarshall

Added two new icons in design

  • arrow-double-left
  • arrow-double-right

*Figma link

📎 Icons sprite file (this sprite contains both icons above)

Reference
@georgewrmarshall 's guide for Export icons as optimized svgs for code
I already went thru the guided steps and optimized, exported the sprite file, which is what u see above.

@georgewrmarshall
Copy link
Contributor Author

@garrettbear you will have to extract the paths and create 2 new svgs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-UI Relating to the user interface. team-design-system All issues relating to design system in Extension
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants