Skip to content
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

Le menu principal ne s'affiche pas avec la version ordinateur sur mobile #6014

Open
AmauryCarrade opened this issue Jan 4, 2021 · 4 comments
Labels
C-Front Concerne l'interface du site S-BUG Corrige un problème

Comments

@AmauryCarrade
Copy link
Member

AmauryCarrade commented Jan 4, 2021

Bonjour !

Il m'arrive d'utiliser ZdS sur mobile, avec l'option « Version pour ordinateur » activée. Cependant, dans ce cas, le menu en haut à gauche composé de 3 lignes horizontales ne fonctionne pas : cliquer dessus n'a aucun effet. En revanche, le menu du profil tout à droite fonctionne bien.

menu bug.png

menu ok.png

Détails techniques

  • JavaScript est bien activé
  • Google Chrome 87.0.4280.101 sur Android
  • Il n'y a pas d'extension ni d'userscript sur ce navigateur

J'ai rapidement cherché et il me semble que ce bug n'a pas encore été remonté.
Désolé si c'est le cas. Merci et bonne année 2021 ! :)

Sujet : https://zestedesavoir.com/forums/sujet/14918/menu-principal-qui-ne-saffiche-pas-avec-la-version-ordinateur-sur-mobile/
Envoyé depuis Zeste de Savoir

@AmauryCarrade AmauryCarrade added C-Front Concerne l'interface du site S-BUG Corrige un problème labels Jan 4, 2021
@AmauryCarrade
Copy link
Member Author

AmauryCarrade commented Jan 4, 2021

Il semblerait que ça vienne d'une détection du navigateur basée sur l'User Agent au lieu de la largeur de l'écran (ou de la visibilité ou non du bouton du menu mobile).

// Android Mobile
var isAndroidMobile = navU.indexOf('Android') > -1 && navU.indexOf('Mozilla/5.0') > -1 && navU.indexOf('AppleWebKit') > -1
// Android Browser (not Chrome)
var regExAppleWebKit = new RegExp(/AppleWebKit\/([\d.]+)/)
var resultAppleWebKitRegEx = regExAppleWebKit.exec(navU)
var appleWebKitVersion = (resultAppleWebKitRegEx === null ? null : parseFloat(regExAppleWebKit.exec(navU)[1]))
var disableMobileMenu = isAndroidMobile && appleWebKitVersion !== null && appleWebKitVersion < 537
if (disableMobileMenu) { $('html').removeClass('enable-mobile-menu') }

A minima, l'affichage du menu au toucher sur ce bouton ne devrait pas être conditionné par l'UA, mais devrait être fonctionnel dans tous les cas (au pire, le bouton n'est pas visible et le code ne sera jamais exécuté). L'ouverture par glissement du doigt pourrait rester conditionnée, si rien de propre n'est trouvé, c'est moins gênant selon moi.

@viki53
Copy link
Contributor

viki53 commented Jan 4, 2021

La détection de l'UA c'est souvent à double tranchant. Il vaut mieux détecter si une fonctionnalité (comme le touch) est supportée directement dans la mesure du possible.

@AmauryCarrade
Copy link
Member Author

AmauryCarrade commented Jan 5, 2021

Il est vrai que pour le glissement, la détection de la capacité de touch c'est encore mieux (et même idéal, en fait, sauf peut-être dans le cas d'ordinateurs avec écran tactiles, encore que).

@viki53
Copy link
Contributor

viki53 commented Jan 5, 2021

On peut cumuler la détection du touch avec une media-query pour filtrer les ordis :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site S-BUG Corrige un problème
Projects
Status: À trier
Development

No branches or pull requests

2 participants