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
#1740 - Improve Autocomplete disabled state #21321
Conversation
Pull Request Test Coverage Report for Build 6ef6080c1494873081f77a72dce3fd2a9f95bf28Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
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.
Thanks! 🙏
Minor issue detected with the double opacity.
I noticed this in Free without Premium, when visiting our Settings > Site representation
and scroll down to the Additional organization info
. The Number of employees
field is an Autocomplete, and you can see it dimmer than the fields above.
Also, the border seems to widen a bit when I select the field still. I couldn't figure out why precisely, but it seems related to the shadow, which is very strange. I'm willing to overlook that 😅
|
||
.yst-autocomplete__button { | ||
@apply | ||
yst-opacity-50 |
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.
Because the button is rendered inside the disabled class, the opacity is doubled. I think you can leave out this class
yst-opacity-50 |
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.
You are right, removing.
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.
CR && AT ✅
Context
Fixes: https://github.com/Yoast/shopify-seo/issues/1740
Summary
This PR can be summarized in the following changelog entry:
Relevant technical choices:
Test instructions
Test instructions for the acceptance test before the PR gets merged
This PR can be acceptance tested by following these steps:
Test for Shopify:
Regression test for Free:
Settings > Site representation
and scroll down to theAdditional organization info
Number of employees
field is not any dimmer than the fields aboveRelevant test scenarios
Test instructions for QA when the code is in the RC
QA can test this PR by following these steps:
Impact check
This PR affects the following parts of the plugin, which may require extra testing:
UI changes
Other environments
[shopify-seo]
, added test instructions for Shopify and attached theShopify
label to this PR.Documentation
Quality assurance
Innovation
innovation
label.Fixes #