-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Minicart thumbnail image showing wrong size in frontend which is half of image size in XML file #35275
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
Comments
Hi @csvikram44. Thank you for your report.
Make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, Add a comment to the issue:
For more details, review the Magento Contributor Assistant documentation. Add a comment to assign the issue: To learn more about issue processing workflow, refer to the Code Contributions.
🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of issues happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket. ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
@magento give me a 2.4-develop instance. |
Hi @csvikram44. Thank you for your request. I'm working on Magento instance for you. |
Hi @csvikram44, unfortunately there is no ability to deploy Magento instance at the moment. Please try again later. |
@magento When will I get the update from your side? |
@magento give me a 2.4-develop instance. |
Hi @csvikram44. Thank you for your request. I'm working on Magento instance for you. |
Hi @csvikram44, unfortunately there is no ability to deploy Magento instance at the moment. Please try again later. |
Hi @engcom-Delta. Thank you for working on this issue.
|
Hi @csvikram44, |
@ihor-sviziev My problem is not with the blurry image. In MiniCart, a thumbnail image of half the size of the image entered in the XML file appears in the MiniCart. File path:
See below screenshot, I added 75px width & height for mini_cart_product_thumbnail in the view.xml file but it shows 37.5px width and due to that it looks small. Instead of showing 37.5px, It should be the 75px width that is entered in the view.xml file. |
@csvikram44 right, you should put here 2x size of the image = 150 |
@ihor-sviziev But why do I need to load a double-size thumbnail image? If I add 10 items to the mini cart then the page load will increase because it loads twice the size of images which is not correct. Also, it was running fine in the old version. Is there any other reason behind doubling the image size? |
@ihor-sviziev I hope you understand my point properly. In my view, this is an issue and because of this, all developers will have to face this problem whenever they upgrade from the lower to the latest Magento version. |
@csvikram44 the reason for that - retina/4k displays that becoming more and more popular everywhere both in mobile and on computers now. |
@ihor-sviziev Thanks for the update. I checked by uploading different images, and none of the images are getting blurred. But there are a lot of chances of loading issues increasing in speed on mobile when multiple product items are added. Please think about it one more time and update me. |
Hi @csvikram44 , View.xmlHence , issue has been confirmed. |
✅ Jira issue https://jira.corp.magento.com/browse/AC-2808 is successfully created for this GitHub issue. |
✅ Confirmed by @engcom-Delta. Thank you for verifying the issue. |
@magento I am working on this |
Hi @olegpola! 👋 |
@magento add to contributors team |
@magento I am working on this |
…d which is half of image size in XML
Hi @csvikram44 You guys had this issue because you declared width and height for Conclusion: The correct solution is double the size of height and width for |
Can somebody comment on the status of this issue? Some say this is a valid issue and others say it is not. Imho the need to double the size in view.xml just to get back to how things were before seems a somewhat strange solution to the issue description. |
@Rickertje This issue should be closed. The need to double the size in view.xml is for fixing issue #24735. If you are still confused, please read my comment above (I've just updated it to say the change in order to fix #24735). |
Hi @csvikram44, |
Hi @ihor-sviziev. Thank you for working on this issue.
|
After upgrading Magento version from 2.3.5-p1 to 2.4.3-p1, I am facing the problem of minicart thumbnail size. I found that in the view.xml file, the width and height of half the size set for the minicart thumbnail shows in the frontend minicart.
Note: It works fine in my old Magento version that the size which is in view.xml file shows the width and height of the size in the frontend.
Preconditions (*)
Steps to reproduce (*)
Important: By the way, this is the default issue in both Blank and Luma themes.
<image id="mini*cart_product*thumbnail" type="thumbnail"> <width>75</width> <height>75</height> </image>
Expected result (*)
Actual result (*)
Please provide Severity assessment for the Issue as Reporter. This information will help during Confirmation and Issue triage processes.
The text was updated successfully, but these errors were encountered: