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

240627 - Icon Updates #2358

Closed
6 tasks done
ratinsl opened this issue Jun 28, 2024 · 6 comments
Closed
6 tasks done

240627 - Icon Updates #2358

ratinsl opened this issue Jun 28, 2024 · 6 comments

Comments

@ratinsl
Copy link

ratinsl commented Jun 28, 2024

Description

We have some icon additions and updates. There are some payment card icons here, and the stroke-width line was added to them. The alipay cn icon from the last batch had the incorrect blue color.

Web skin name mapping doc has been updated with the below items.

[New icons]

  • Chevron down (20)

[Updated icons]

  • Alipay cn (32, 24, 18, 12) -colored

Thank you, and let me know if you have any questions,
Ryan

242724_Web_Icons.zip

Screenshots

No response

Figma link

No response

Engineering pre-requisites

  • Successful design review with Design System Team.
  • Supports dark mode.
  • Supports responsive design.
  • Tokens have been provided (or already exist).
  • Successful accessibility review with Core A11Y Team.
  • Successful frontend review with eBayUI Team.
@ArtBlue
Copy link
Contributor

ArtBlue commented Jul 1, 2024

Hey @ratinsl , I noticed you filed this late last week. We had already done the planning for the next release and already have an icons issue (not started yet - #2347) slated for next release. The Allipay icons are also included in that issue. To avoid confusion and get these updated into the next release as well, it might be best to reconcile these icons and move the changes from these two icons into that issue and we can just close this one.

@ratinsl
Copy link
Author

ratinsl commented Jul 1, 2024

Hey @ArtBlue, created a new ticket because I thought the other one had already been picked up, but that's great - I'll add these to the other one later today!

@ArtBlue
Copy link
Contributor

ArtBlue commented Jul 1, 2024

@ratinsl , maybe you're not seeing the statuses??? I know we've had this issue where you weren't seeing everything on the issue. Can you see this on the right side on #2347 ?

image

@ratinsl
Copy link
Author

ratinsl commented Jul 3, 2024

Closing this ticket - added these items to the other open one. @ArtBlue I saw the part under "projects" but instead of "ToDo" I thought it was already assigned to a project which is why I created a new ticket. Will look out for this next time, thanks!

@ratinsl
Copy link
Author

ratinsl commented Jul 3, 2024

Closing this one - added these items to the other open one. @ArtBlue I saw the part under "projects" but instead of "ToDo" I thought it was already assigned to a project which is why I created a new ticket.

@ratinsl ratinsl closed this as completed Jul 3, 2024
@ArtBlue
Copy link
Contributor

ArtBlue commented Jul 8, 2024

Np, @ratinsl . It is assigned to a project. The project is eBayUI, but the status field is a finer grained indication of status. With the exception of when we forget to move issues properly along the dev cycle, those should reflect the current status of the actual dev work.

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

No branches or pull requests

2 participants