Prestashop: Always showing mobile menu on v1.7.6.3

[josedvd123]

Describe the bug
I've just installed a new PrestaShop Store and when I got to the admin panel the mobile menu is always shown, even on a desktop and with fullscreen. My screen resolution is 1920 x 1080.

To Reproduce
Steps to reproduce the behavior:

1. Go to admin panel
2. See error

Screenshots

Additional information
PrestaShop version: 1.7.6.3
PHP version: 7.2

[khouloudbelguith]

Hi @josedvd123,

Thanks for your report.
I manage to reproduce the issue with PS1.7.6.0 & PS1.7.6.1 & PS1.7.6.2 & PS1.7.6.3.
I tried with Ipad4 & Ipad Mini 2

1. In the Dashboard Page, Orders page (not migrated page)
   The menu mobile is always displayed & when we click on it => nothing happens (it should display or not the menu)
   
   with PS1.7.5.2 => OK
   
2. In the products page (Migrated page) the menu is not well displayed => some items cannot be selected.
   

with PS1.7.5.2 => OK

I’ll add this to the debug roadmap so that it’s fixed. If you have already fixed it on your end or if you think you can do it, please do send us a pull request!
Thanks!

[josedvd123]

Sorry, it was a conflict with a Chrome extension called "Letyshops".
I just disabled the extension and now all is fine.
Thanks

[khouloudbelguith]

@josedvd123, I will re-open the issue.
Issue reproduced with Ipad4 & Ipad Mini 2 using the https://www.browserstack.com.
https://drive.google.com/file/d/1z8xIZDZbpOcwK0A8UCdzUE0S1LmvkZ-T/view
Using the same device with PS1.7.5.2 => OK.
https://drive.google.com/file/d/1kDnofITgc-7CowHaWHNkv2-eewC-BHtg/view

Thanks!

[NeOMakinG]

@khouloudbelguith , we follow Boostrap browser support (https://getbootstrap.com/docs/4.0/getting-started/browsers-devices/) it means iOS7 is not supported as Safari on this iOS doesnt support css flex feature and more, it's not a bug, peoples using an iPad 4 or mini 2 should be on iOS 9 mini to be compatible, which is not even the latest version supported on these devices

(tested on ios 9 and it looks fine)

[khouloudbelguith]

Hi,

As discussed with @NeOMakinG.

1. When using iOS7 => issue reproduced only with PS1.7.6
   
   With PS1.7.5.2 => Ok (regression)
   

1. When using iOS9 => issue reproduced also with PS1.7.5.2 & PS1.7.6 (not a regression)
   
   

2. When using iOS10 => issue not reproduced
   

3. When using iOS10 => other bug reported fixed by this PR: https://github.com/PrestaShop/PrestaShop/pull/17459

Bootstrap supports the latest, stable releases of all major browsers and platforms. On Windows, we support Internet Explorer 10-11 / Microsoft Edge.

Ping @PrestaShop/prestashop-product-team @PrestaShop/prestashop-core-developers what do you think of this issue.
Do we need to support only iOS10 minimum?

Thanks!

[PierreRambaud]

IHMO, we should ignore old browsers

[Progi1984]

What is an old browser ?

Bootstrap support the latest, stable release of browser... but which version ? Now ? or at the release date of the version of Bootstrap ?

[PierreRambaud]

Depending on the bootstrap release we used.

[colinegin]

I quite agree with other comments, i'm not in favor of working on it as it represents very very very few users (0,70% worldwide for ios10 and previous versions). But i'm still wondering why this used to be working on ios 7 with PS 1.7.5.2, do you guys have any idea ? @PrestaShop/prestashop-core-developers

[NeOMakinG]

I just checked on 1.7.5.x, it's buggy too due to the use of display flex which is supported on the safari version of ios 10

There is no regression between 1.7.5.x and 1.7.6.x+

And we should not support browsers that are not supporting display flex (browsers < 2015 except IE11)

[khouloudbelguith]

Hi @NeOMakinG,

I tried with iOSv7 & PS1.7.5.2 & it is OK
https://drive.google.com/file/d/1WMPM6S97WFiA7QzFMpuxIIePOPhlBjoN/view
Using the same device with PS1.7.6 => NOK
https://drive.google.com/file/d/1rnFEVRGx5t5r4Abf8W85S7mQ1dlFnYU_/view
I think this is a regression.

Thanks!

[NeOMakinG]

My bad I was testing wrong, but I should reinstall prestashop on this branch to test, anyway it's also buggy on 1.7.7.x so I see what's wrong and it's flex support and more, which is something used everywhere, we need to support it, browsers not supporting it are way too outdated