-
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
Textarea patch 1 #26142
Textarea patch 1 #26142
Conversation
Hi @textarea. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
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 we should rather change the selector in JavaScript file to [data-role="subtitle"]
as I wasn't able to find camel case data-role
attribute value anywhere else in Magento.
Hi @krzksz, as you prefer, but there are different camel case data-role attributes, for example: |
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.
That's true I don't know why I didn't find it when searching through the files.
Hi @krzksz, thank you for the review.
|
Hi @textarea, great contribution! |
I will take care of test coverage |
…o textarea-patch-1
Thanks @Nazar65 ! |
e434332
to
fcce5b8
Compare
✔️ QA Passed |
failed Functional Tests B2B Not related to the changes in this PR |
Hi @krzksz, thank you for the review. |
@textarea Please fix failing Unit test |
I will fix them |
@slavvka done ✔️ |
Hi @textarea, thank you for your contribution! |
Description (*)
On modal setTitle() call, subTitle tag get erased and not appended any more, caused by:
modalSubTitle: '[data-role="subTitle"]'
Fixed Issues (if relevant)
Manual testing scenarios (*)
Contribution checklist (*)