-
Notifications
You must be signed in to change notification settings - Fork 33
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/hover and link styles minor issues #1844
Fix/hover and link styles minor issues #1844
Conversation
Bundle Size Diff
|
Plugin build for 108a699 is ready 🛎️!
|
E2E SummaryTyping
Values above 60ms"0 - 67.80, 13 - 66.29, 16 - 61.89" |
Here are the details from a test instance, @JohnPixle , in case you have time and also want to test the changes. https://irritatehorses.s4-tastewp.com/wp-admin |
Hey @mghenciu, When you're done with the changes, don't forget to ask Robert or me for a PR review. One thing I wanted to mention is if you're making style changes to core components like buttons, I'll recommend you to test with previous versions of WordPress to ensure it doesn't affect them negatively. Thanks! |
Thank you for the feedback, Hardeep. Do you mean to test in the Otter Dashboard context? or also in other places? |
@mghenciu Only testing in Dashboard should be enough. |
I did tested with:
And the style changes look ok. Let me know if something else is required @HardeepAsrani , before asking for a review on this. |
@mghenciu That's great then, you can put it to a review & then QA. |
@Soare-Robert-Daniel Can you review the changes here? |
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.
Tested and looks good.
@mghenciu everything's working well here 👍 |
🎉 This PR is included in version 2.4.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Closes #1843 .
Summary
Minor styles changes for hover, focus, links and primary accent colors.
1. Added hover state for the Navigation panels, since we didn't had one
Screen.Recording.2023-09-01.at.13.29.46.mov
2. Improved the hover state for the Accordion Panels, to increase the contrast.
3. Made the toggles, Secondary buttons and link - to use 1 single color. In the current live version, those colors change when changing the theme, which creates inconsistent results (attached below):
Test instructions
Checklist before the final review