-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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 icon advance attributes properties not change #28032
Fix icon advance attributes properties not change #28032
Conversation
Hi @mrtuvn. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
@magento give me test instance |
Hi @edenduong. Thank you for your request. I'm working on Magento instance for you |
Hi @edenduong, here is your new Magento instance. |
@magento give me test instance |
Hi @edenduong. Thank you for your request. I'm working on Magento instance for you |
Hi @edenduong, here is your new Magento instance. |
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.
&.active { | ||
&:before { | ||
content: @icon-expand-open__content; | ||
content: @icon-expand-close__content; |
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.
I think not active and active can not have the same content : "close". And this issue #22702 will occur again if you edit this line from "open" to "close"
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.
Hi @edenduong after see issue 22702 i just found another problem exist!. The class opened already hardcoded in html and seem not work properly as expected. If you click on title that class not change or removed
Here is reference
https://github.com/magento/magento2/blob/2.4-develop/app/code/Magento/ConfigurableProduct/view/adminhtml/web/template/variations/steps/summary-grid.html#L8
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.
If not active available content open will always used
See this
https://github.com/magento/magento2/blob/2.4-develop/app/design/adminhtml/Magento/backend/Magento_Backend/web/css/source/module/main/_collapsible-blocks.less#L75
Hi @edenduong, thank you for the review. |
…perties-not-change
Hi @mrtuvn, thank you for your contribution! |
Description (*)
Previously icon not change after user click on fieldset to expand content
Normal behavior icon will change to indicator user content expanded. Just like product form fieldset
After this patch icon changed correctly as expected
Related Pull Requests
Fixed Issues (if relevant)
Summary grid in modal when configurable product create
Recheck
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)