Skip to content
This repository has been archived by the owner on Dec 6, 2022. It is now read-only.

Add double chevron icons #811

Merged
merged 2 commits into from
May 17, 2021
Merged

Add double chevron icons #811

merged 2 commits into from
May 17, 2021

Conversation

tjmw
Copy link
Member

@tjmw tjmw commented May 7, 2021

What is the purpose of this change?

Desire to use double chevron icons in DCR.

What does this change?

Adds double chevron icons.

Screenshots

Screenshot 2021-05-07 at 11 56 13

Checklist

Accessibility

Cross browser and device testing

  • Tested with touch screen device

Responsiveness

  • Tested at all breakpoints
  • Tested with with long text
  • Stretched to fill a wide container

Documentation

  • Full API surface area is documented in the README
  • Examples in Storybook

Known issues

@tjmw @abeddow91
Add double chevron icons
38982c8 
Co-authored-by: Anna Beddow <anna.beddow@guardian.co.uk>
@tjmw tjmw requested a review from a team as a code owner May 7, 2021 10:57
@probot-autolabeler probot-autolabeler bot added the Icons Changes to icons label May 7, 2021
SiAdcock
SiAdcock approved these changes May 17, 2021
View reviewed changes
Copy link
Contributor

@SiAdcock SiAdcock left a comment

Choose a reason for hiding this comment

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

Looks great, thanks so much @tjmw 💎

@SiAdcock SiAdcock merged commit 7e683f2 into main May 17, 2021
@SiAdcock SiAdcock deleted the add-double-chevron-icons branch May 17, 2021 09:49
@SiAdcock SiAdcock mentioned this pull request May 20, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@SiAdcock SiAdcock SiAdcock approved these changes

Assignees
No one assigned
Labels
Icons Changes to icons
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tjmw @SiAdcock