-
Notifications
You must be signed in to change notification settings - Fork 9.4k
[fixed My Wish List Product not showing properly between >768px and <… #26546
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
[fixed My Wish List Product not showing properly between >768px and <… #26546
Conversation
Hi @hitesh-wagento. 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.
Hello, @hitesh-wagento, thank you for your contribution.
I don't understand why did you choose those values for margin and width. Also later on, they are overwritten by width: 100% and margin: 0 anyway, aren't they?
Hi @ptylek I have applied margin as per desktop and >640px; and after <640px you can see 100% width not required margin. Thanks |
Okay, I understand @hitesh-wagento. But why these values such as margin-left: 2%; padding: 5px; width: calc(~'(100% - 4%) / 3')? I couldn't find any other grid using these values. Aren't there any variables? |
Hi @ptylek You can find this variable in category pages on a smaller size in the below CSS.
Thanks |
app/design/frontend/Magento/blank/Magento_Wishlist/web/css/source/_module.less
Outdated
Show resolved
Hide resolved
app/design/frontend/Magento/blank/Magento_Wishlist/web/css/source/_module.less
Outdated
Show resolved
Hide resolved
app/design/frontend/Magento/blank/Magento_Wishlist/web/css/source/_module.less
Outdated
Show resolved
Hide resolved
app/design/frontend/Magento/luma/Magento_Wishlist/web/css/source/_module.less
Outdated
Show resolved
Hide resolved
app/design/frontend/Magento/luma/Magento_Wishlist/web/css/source/_module.less
Outdated
Show resolved
Hide resolved
app/design/frontend/Magento/luma/Magento_Wishlist/web/css/source/_module.less
Outdated
Show resolved
Hide resolved
app/design/frontend/Magento/luma/Magento_Wishlist/web/css/source/_module.less
Show resolved
Hide resolved
app/design/frontend/Magento/blank/Magento_Wishlist/web/css/source/_module.less
Show resolved
Hide resolved
@hitesh-wagento, thanks for clarification, I added some comments |
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.
Great, @hitesh-wagento! Thank you for your contribution.
Hi @ptylek, thank you for the review. |
✔️ QA Passed |
#mm20in |
Hi @hitesh-wagento, thank you for your contribution! |
Description (*)
Fixed My Wish List Product not showing properly between >768px and <1023px
Fixed Issues (if relevant)
Manual testing scenarios (*)
Expected result
Products should show 3 items

Actual result
Products show 2 products and space right side
Contribution checklist (*)