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

Tag - Update Figma component to use "Close" icon. #3467

Closed
4 tasks done
ashifalinadaf opened this issue Jul 10, 2023 · 8 comments
Closed
4 tasks done

Tag - Update Figma component to use "Close" icon. #3467

ashifalinadaf opened this issue Jul 10, 2023 · 8 comments
Assignees
Labels
Owner:Design Used in issues that are being worked on/should be worked on by a designer.

Comments

@ashifalinadaf
Copy link
Contributor

ashifalinadaf commented Jul 10, 2023

Replacing the close icon for the Tag from 'form icon' to 'core icon'
Related ticket - #3256

Deliverables:

@ashifalinadaf
Copy link
Contributor Author

ashifalinadaf commented Jul 11, 2023

Created a branch and updated all the 'Close Icons' from 'form close icon' to 'core close icon'.

@kenny If you can a look and let me know if there are any issues.

@halocline halocline added the Owner:Design Used in issues that are being worked on/should be worked on by a designer. label Aug 1, 2023
@vavalos5
Copy link
Collaborator

vavalos5 commented Aug 2, 2023

@ashifalinadaf I remember taking a look at your branch and seeing that the icons had been replaced. I can take this on and merge it over and create a subsequent ticket for it to be added to grommet.

@vavalos5 vavalos5 self-assigned this Aug 2, 2023
@ashifalinadaf
Copy link
Contributor Author

ashifalinadaf commented Aug 2, 2023

Sure @vavalos5 thank you for reminding this.

@vavalos5
Copy link
Collaborator

vavalos5 commented Aug 2, 2023

@KennyAtHPE @taysea Can both of you take a look at the branch and tell me if you notice any of the tags using the incorrect "close icon" please. Thanks!

@taysea
Copy link
Collaborator

taysea commented Aug 3, 2023

It looks like the correct icon is being used. Also, if it's a quick fix (<5min) can the page be named "Tag" instead of "Tags"? If not a quick fix, can a ticket be filed to align with the Grommet component name?

@vavalos5
Copy link
Collaborator

vavalos5 commented Aug 3, 2023

@taysea Yea, good call out. I'll make those changes today.

@KennyAtHPE
Copy link
Collaborator

@vavalos5 can confirm that all the variants are using the correct 'core / close' icon.

@vavalos5
Copy link
Collaborator

vavalos5 commented Aug 3, 2023

Icons have been merged to main DS file and published. In addition, the file name was changed from "Tags" to "Tag"

@vavalos5 vavalos5 closed this as completed Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Owner:Design Used in issues that are being worked on/should be worked on by a designer.
Projects
None yet
Development

No branches or pull requests

5 participants