-
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
Cannot slide images in product page in mobile in 2.4.6 only! #37232
Comments
Hi @cptX. Thank you for your report.
Join Magento Community Engineering Slack and ask your questions in #github channel. |
Can you record video mobile show your web ? android or ios firms |
Hi @engcom-Dash. Thank you for working on this issue.
|
@magento give me 2.4.6 instance |
Hi @engcom-Dash. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Dash, here is your Magento Instance: https://dd3ebb1b87e8648213f0060c16627297.instances-prod.magento-community.engineering |
@magento give me 2.4-develop instance |
Hi @engcom-Dash. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Dash, here is your Magento Instance: https://cfe24c18561a2f21c44d921f8db2c17e.instances-prod.magento-community.engineering |
@mrtuvn hi, I don't have another camera available to record a video on the physical phone but anyway there is no point to show a video. If you just visit the product page in a mobile phone you cannot slide right or left the images of the product. The first image stays frozen on the screen and this is MAJOR BUG. |
so can you share you website to product link accessible at version 2.4.6 ? It's will easy and faster way to figure exactly problem inmho |
@mrtuvn I don't understand your request. As you see above a new magento instance has been created. You just have to visit it. For your convenience check this product in mobile view with touch enabled or just from your mobile phone |
I was not aware of this group. Tried to register but it says the invitation sent to me was expired. How can I join this group? |
yes i see your point here |
This is major bug and I'm glad that you verified this! |
Please provide a patch asap, I need to publish my site next week and this is a major blocking issue!!! |
As a fast fix, you can revert the following commit: d1c5ccf |
@magento give me 2.4.6 instance |
Hi @engcom-Dash. Thank you for your request. I'm working on Magento instance for you. |
Can report this is also happening to me after I upgrade my site yesterday from 2.4.5 to 2.4.6. |
Hi @engcom-Dash, here is your Magento Instance: https://dd3ebb1b87e8648213f0060c16627297.instances-prod.magento-community.engineering |
@roma84 I tried to implement this fix suggested but still not working on my site. |
It worked for me on 2.4.6. You need to clear and re-generate static content after the change. Maybe the following fix is correct for line 1143 in file fotorama.js: el.addEventListener ? el.addEventListener(e, fn, {passive: !!bool}) : el.attachEvent('on' + e, fn); |
Hello @MagePsycho, We have again checked this issue in the test instance containing latest development branch ie 2.4-develop and the issue is not reproducible for us. Please refer to the below screencast for reference: WhatsApp.Video.2024-03-18.at.1.07.33.PM.mp4The related JIRA shows the fixed version of this issue as 2.4.7-beta1. Thanks |
@engcom-Hotel I can see this issue in Magento |
I am using 2.4.6-p4 and It was fixed after this change. |
Thanks, in magento 2.4.6-p6 also solved the sliding problem in mobile! |
Due to the latest update (Magento 2.4.7-p1), we have taken another look at the corrections. The adjustments fix the problem with swiping. However, the addEvent method was not adapted correctly. The current structure of the method (in Magento 2.4.7-p1) looks like this: function addEvent(el, e, fn, bool) {
if (!e) return;
el.addEventListener ? el.addEventListener(e, fn, {passive: !!bool}) : el.attachEvent('on' + e, fn);
} The 4th parameter ("bool") is currently forwarded to the "passive" option. This was not originally intended. See the "addFocus" method (in Magento 2.4.7-p1): function addFocus(el, fn) {
addEvent(el, 'focus', el.onfocusin = function (e) {
fn.call(el, e);
}, true);
} As already described in the previous comment, the "addFocus" method already existed before the "passive" option was introduced. The intention of addFocus or the call to addEvent was therefore to use the "capture" option with the 4th parameter. It was also an excellent suggestion to set the "passive" option based on the EventType ("wheel"). The addEvent method should therefore be structured as follows: function addEvent(el, e, fn, bool) {
if (!e) return;
const options = {
get passive() {
return e === 'wheel';
}
get capture() {
return !!bool;
}
}
el.addEventListener ? el.addEventListener(e, fn, options) : el.attachEvent('on' + e, fn);
} |
@magento give me 2.4-develop instance |
Hi @engcom-Dash. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Dash, here is your Magento Instance: https://cfe24c18561a2f21c44d921f8db2c17e.instances-prod.magento-community.engineering |
Hi @cptX Thanks for reporting and collaboration. I verified the issue on both Magento 2.4 dev instance and Magento 2.4.7-p1. The issue is not reproducible. Able to slide images on product page on mobile Please refer to the attached screen recording 37232-1.mov |
For me it does work in Chrome "mobile" mode as displayed in the previous post's video, but it doesn't work on real mobile devices running Chrome. |
@magento give me 2.4-develop instance |
Hi @engcom-Bravo. Thank you for your request. I'm working on Magento instance for you. |
Hi @engcom-Bravo, here is your Magento Instance: https://cfe24c18561a2f21c44d921f8db2c17e.instances-prod.magento-community.engineering |
Hi @DannyDaemonic, Thanks for your update. We have verified the issue in Latest 2.4-develop instance using real mobile device using chrome and the issue is no more reproducible.Kindly refer the attached video. Screenrecorder-2024-07-08-12-28-11-470.mp4We are able to slide the images in a mobile device. Hence we are closing this issue. Thanks. |
@engcom-Bravo Thanks for taking a look. Not to be pedantic, but were you sliding the image with your finger or were you tapping on the dots at the bottom? The feature that was lost wasn't the use of the dots below the image, but the actual dragging of the image with your finger. |
Hi @DannyDaemonic, Thanks for your update. We have tried with the fingers for sliding the images in a mobile device.Kindly let us know if we are missing anything. Thanks. |
Found the mobile interaction with fotorama gallery to be an issue Magento ver. 2.4.6-p6 Following the threads on this issue the following commit resolves the issue: 9d751c5#diff-fb7ac60e407ee78fc8e5ff56eef13de3f3bbe8a8ce52c14d674f0015f502e7c7 |
Preconditions and environment
Luma Theme
Steps to reproduce
Create a product with more than one image, 3 or 4 would be perfect!
In the product page, using a mobile or mobile view try to slide images to the left (or to the right) with your finger!
Images are not sliding and not changing at all, the slider is not responding. Only way to see the next image is to click on the tiny dot under the image.
I consider this major bug (as usual unexplained regression once again...)
Expected result
The images should slide following the finger of the user!
Actual result
Only the first image of the product is displayed and user is completely unable to view the rest images! This is major bug!!!
Why magento is doing so often such basic regressions? Why simple things that used to work correctly are breaking down the road in every release??
Additional information
No response
Release note
No response
Triage and priority
The text was updated successfully, but these errors were encountered: