-
Notifications
You must be signed in to change notification settings - Fork 60
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 finding child content-type elements in stage-builder #869
base: develop
Are you sure you want to change the base?
Conversation
f34d6de
to
1bfc575
Compare
@magento run all tests |
@magento create issue |
Hello @orukusaki, ❌ QA not Passed We have tried reproducing the issue by following the Please refer to the below custom module we have made to reproduce the issue: Please let us know if we missed anything. Thanks |
Hello @orukusaki, We are still waiting for your reply on this PR. Thanks |
Hi, |
@magento run all tests |
Hello @orukusaki, Thanks for the update on this PR! ✔️ QA Passed Manual testing scenario:
Tested all the manual scenarios, no impact on regression testing. Meanwhile the tests are running on this PR, moving it to Thanks |
@magento run Functional Tests CE, Functional Tests EE, Functional Tests B2B, Performance Acceptance Tests |
@magento run Functional Tests B2B |
@magento run Functional Tests B2B |
Description (*)
Fixes a bug whereby child content-type elements will not be found, where other types of elements also exist at the same level.
Manual testing scenarios (*)
master.html:
preview.html:
It is possible to create this content type in Pagebuilder, and the correct result is rendered into the database and on the frontend. However, Pagebuilder cannot then load up the content back into the stage - the presence of the
<li>
element causes only the last item to be found, and<div class="swiper-scrollbar">
causes even that to be forgotten.This small fix corrects this, and all types of content can now be loaded into the stage correctly.
Checklist
Resolved issues: