-
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
Image in minicart is blurred on iPhone #24735
Comments
Hi @ihor-sviziev. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. @ihor-sviziev do you confirm that you were able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Hi @ajijshekh123. Thank you for working on this issue.
|
Give me @magento give me 2.3-develop instance |
Hi @ajijshekh123. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
Hi @ajijshekh123, here is your Magento instance. |
@ajijshekh123 Thank you for verifying the issue. Unfortunately, not enough information was provided to acknowledge ticket. Please consider adding the following:
Once all required information is added, please add label |
@ajijshekh123 Thank you for verifying the issue. Unfortunately, not enough information was provided to acknowledge ticket. Please consider adding the following:
Once all required information is added, please add label |
✅ Confirmed by @ajijshekh123 Issue Available: @ajijshekh123, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself. |
@magento I'm working on it |
Hi @kovinka. Thank you for working on this issue.
|
Hi @ihor-sviziev. Thank you for your report.
The fix will be available with the upcoming 2.4.0 release. |
This was a terrible fix! Now, with the width/2 code, you just get a teeny tiny image in a big white space. The correct solution would be to put an image that's LARGER than the 75 pixel container into the space to accommodate the @2x resolution. |
I'm too getting half sized images in the minicart on desktop now. was this tested? |
We discovered that it was partly our fault. The newer version of Magento updated the view.xml file to a value of 150 and then the .phtml file divides it (value from XML) in half. We had modified the view.xml and still had the old value that was then divided by the phtml to result in tiny images. If you haven't modified the xml or .phtml, then Magento works correctly "out of the box" but if you modified one or the other, you need to integrate the change into your code. the XML: we had values of 75 (the old values). We removed this XML node to fix our problem, or you could change it to 150 which is what Magento uses by default now. |
Preconditions (*)
Steps to reproduce (*)
Expected result (*)
Actual result (*)
The text was updated successfully, but these errors were encountered: