-
-
Notifications
You must be signed in to change notification settings - Fork 432
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
fix(page): added delete-item hover color #4299
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
but our design: I think an icon needs to be added here cc @zanwei |
I have added the delete icon |
The delete icon and the edit icon, check detail here |
default status, hover status are also missing, don't forget the blue shadow in the active status |
default status, hover status are also missing, don't forget the blue shadow in the active status |
The menu UI refactoring is not yet complete, I think we just need to simply fix the UI of delete for now, and I'll finish the menu UI refactoring in the next iteration. |
Yes, @xell I think we need to distinguish the |
I guess, I have done that, Kindly check it and if you need any changes then let me know |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As @zzj3720 said approve this issue firstly
fixed: #4298