-
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 issue thumbnail image too big in order summary checkout #28644
Fix issue thumbnail image too big in order summary checkout #28644
Conversation
Hi @mrtuvn. Thank you for your contribution
❗ Automated tests can be triggered manually with an appropriate comment:
You can find more information about the builds here ℹ️ Please run only needed test builds instead of all when developing. Please run all test builds before sending your PR for review. For more details, please, review the Magento Contributor Guide documentation. |
Hello @mrtuvn Thank you for the fix! |
173e032
to
88e908b
Compare
@VladimirZaets can you recheck review |
@magento run all tests |
Hi @mrtuvn. Thanks for collaboration. I reran the build but due to changes only in styles I can approve it right now |
Hi @VladimirZaets, thank you for the review.
|
f6bcbb1
to
80d0157
Compare
@magento run all tests |
@VladimirZaets Hi any update on this |
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.
@VladimirZaets Could you review last commit?
@engcom-Delta is this commit in #28546 has cherry pick or merge to 2.4-develop ? |
Hi @VladimirZaets, thank you for the review.
|
@magento run all tests |
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.
Pull Request state was updated. Re-review required.
revert code fix test fail
@engcom-Delta here is my screen in local setup with latest commit show summary area |
80d0157
to
ddf3122
Compare
@engcom-Delta just synced my PR branch with 2.4-develop code |
@magento run all tests |
Hi @mrtuvn. Thanks for collaboration. The issue is closed due that it was fixed by internal team in the scope of 2.4.0 release. The details and screenshots you can see in the issue. Feel free to reopen your PR if the issue is still reproducible |
Hi @mrtuvn, thank you for your contribution! |
Description (*)
This fix bring the fix for image thumbnail in order summary area in checkout page
Previous image too big visible
#1 Recheck styles visual for all (mobile | ipad | desktop) view
since this review #20856 (review) from @VladimirZaets
Related Pull Requests
#20856
Fixed Issues (if relevant)
Manual testing scenarios (*)
1.Add any product type to cart
2.Go to checkout page
3.Open area order summary
Questions or comments
Contribution checklist (*)