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

Réécrit le JS des dropdowns #4454

Merged
merged 5 commits into from Aug 7, 2017

Conversation

Projects
None yet
4 participants
@motet-a
Member

motet-a commented Aug 5, 2017

Le menu principal s’ouvre au survol quand l’option est activée.

J’ai essayé d’améliorer la navigation au clavier en permettant de
basculer entre les menus avec les flèches de gauche et de droite. La
navigation avec reste identique.

Désolé, ce commit n’est pas très atomique. Notez que j’ai retiré
l’engrenage apparaissant en haut à droite pour les utilisateurs
simple, qui faisait un peu doublon avec l’image de profil. Je pense
que de nos jour, la plupart des gens iront naturellement chercher les
réglages de leurs compte en cliquant sur leur avatar en haut à
droite : Google, GitHub et pleins d’autres sites fonctionnent comme ça.

Sur mobile, cette image de profil en haut à droite disparaît, mais
toutes les options du menu apparaissent dans le menu “hamburger” de
gauche. Il n’y a donc pas de fonctionnalités retirées.

La barre de recherche en haut à droite n’est du coup plus toujours
alignée avec les boutons du dessus. Je pense qu’il faudrait mettre une
barre de recherche qui s’agrandit quand elle prends le focus, c’est
facile à faire mais ça sera une autre PR.

Q R
Type de modification Nouvelle fonctionnalité et évolution
Ticket(s) (issue(s)) concerné(s) #661 (c’est vieux)

QA

Testez sur un maximum de navigateurs, à la souris et au clavier (Tab, Shift + Tab et flèches de droite et de gauche)

Sur IE 11, la navigation avec les flèches de droite et de gauche ne marche pas. Mais je pense qu’on s’en fiche un peu, c’est une fonctionnalité introduite par cette PR.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 5, 2017

Coverage Status

Coverage remained the same at 89.273% when pulling bec8793 on motet-a:le-menu-souvre-au-survol into 233d031 on zestedesavoir:dev.

coveralls commented Aug 5, 2017

Coverage Status

Coverage remained the same at 89.273% when pulling bec8793 on motet-a:le-menu-souvre-au-survol into 233d031 on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 5, 2017

Coverage Status

Coverage remained the same at 89.273% when pulling e43092d on motet-a:le-menu-souvre-au-survol into 233d031 on zestedesavoir:dev.

coveralls commented Aug 5, 2017

Coverage Status

Coverage remained the same at 89.273% when pulling e43092d on motet-a:le-menu-souvre-au-survol into 233d031 on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 5, 2017

Coverage Status

Coverage remained the same at 89.273% when pulling 2703919 on motet-a:le-menu-souvre-au-survol into 233d031 on zestedesavoir:dev.

coveralls commented Aug 5, 2017

Coverage Status

Coverage remained the same at 89.273% when pulling 2703919 on motet-a:le-menu-souvre-au-survol into 233d031 on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 6, 2017

Coverage Status

Coverage remained the same at 88.886% when pulling bf6debc on motet-a:le-menu-souvre-au-survol into c433e6a on zestedesavoir:dev.

coveralls commented Aug 6, 2017

Coverage Status

Coverage remained the same at 88.886% when pulling bf6debc on motet-a:le-menu-souvre-au-survol into c433e6a on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 6, 2017

Coverage Status

Coverage remained the same at 88.878% when pulling 27cef34 on motet-a:le-menu-souvre-au-survol into 482960a on zestedesavoir:dev.

coveralls commented Aug 6, 2017

Coverage Status

Coverage remained the same at 88.878% when pulling 27cef34 on motet-a:le-menu-souvre-au-survol into 482960a on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 6, 2017

Coverage Status

Coverage remained the same at 88.878% when pulling 2907653 on motet-a:le-menu-souvre-au-survol into 482960a on zestedesavoir:dev.

coveralls commented Aug 6, 2017

Coverage Status

Coverage remained the same at 88.878% when pulling 2907653 on motet-a:le-menu-souvre-au-survol into 482960a on zestedesavoir:dev.

Réécrit le JS des dropdowns
Le menu principal s’ouvre au survol quand l’option est activée.

J’ai essayé d’améliorer la navigation au clavier en permettant de
basculer entre les menus avec les flèches de gauche et de droite.  La
navigation avec <tab> reste identique.

Désolé, ce commit n’est pas très atomique. Notez que j’ai retiré
l’engrenage apparaissant en haut à droite pour les utilisateurs
simple, qui faisait un peu doublon avec l’image de profil. Je pense
que de nos jour, la plupart des gens iront naturellement chercher les
réglages de leurs compte en cliquant sur leur avatar en haut à
droite : Google, GitHub et pleins d’autres sites fonctionnent comme ça.

Sur mobile, cette image de profil en haut à droite disparaît, mais
toutes les options du menu apparaissent dans le menu “hamburger” de
gauche. Il n’y a donc pas de fonctionnalités retirées.

La barre de recherche en haut à droite n’est du coup plus toujours
alignée avec les boutons du dessus. Je pense qu’il faudrait mettre une
barre de recherche qui s’agrandit quand elle prends le focus, c’est
facile à faire mais ça sera une autre PR.

Fix #661
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 6, 2017

Coverage Status

Coverage remained the same at 88.878% when pulling 1808531 on motet-a:le-menu-souvre-au-survol into 482960a on zestedesavoir:dev.

coveralls commented Aug 6, 2017

Coverage Status

Coverage remained the same at 88.878% when pulling 1808531 on motet-a:le-menu-souvre-au-survol into 482960a on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 7, 2017

Coverage Status

Coverage increased (+15.8%) to 88.878% when pulling eb1fcc3 on motet-a:le-menu-souvre-au-survol into 9a07f4b on zestedesavoir:dev.

coveralls commented Aug 7, 2017

Coverage Status

Coverage increased (+15.8%) to 88.878% when pulling eb1fcc3 on motet-a:le-menu-souvre-au-survol into 9a07f4b on zestedesavoir:dev.

@pierre-24

This comment has been minimized.

Show comment
Hide comment
@pierre-24

pierre-24 Aug 7, 2017

Member

Rapport de QA: OK (FF, IE, chrome), mais ...

... Mais en fait, ce survol est activé par défaut. J'ai donc peur qu'on aie droit à des plaintes à ce sujet. Je propose donc que tu change la valeur par défaut de ce champ et que tu rajoutes une instructions dans le UPDATE.md disant grosso modo d'exécuter le bout de code suivant dans le shell django (python manage.py shell):

from zds.member.models import Profile
Profile.objects.update(is_hover_enabled=False)
Member

pierre-24 commented Aug 7, 2017

Rapport de QA: OK (FF, IE, chrome), mais ...

... Mais en fait, ce survol est activé par défaut. J'ai donc peur qu'on aie droit à des plaintes à ce sujet. Je propose donc que tu change la valeur par défaut de ce champ et que tu rajoutes une instructions dans le UPDATE.md disant grosso modo d'exécuter le bout de code suivant dans le shell django (python manage.py shell):

from zds.member.models import Profile
Profile.objects.update(is_hover_enabled=False)
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 7, 2017

Coverage Status

Coverage increased (+15.8%) to 88.878% when pulling 262f7c7 on motet-a:le-menu-souvre-au-survol into 9a07f4b on zestedesavoir:dev.

coveralls commented Aug 7, 2017

Coverage Status

Coverage increased (+15.8%) to 88.878% when pulling 262f7c7 on motet-a:le-menu-souvre-au-survol into 9a07f4b on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 7, 2017

Coverage Status

Coverage increased (+15.8%) to 88.878% when pulling 7f85891 on motet-a:le-menu-souvre-au-survol into 9a07f4b on zestedesavoir:dev.

coveralls commented Aug 7, 2017

Coverage Status

Coverage increased (+15.8%) to 88.878% when pulling 7f85891 on motet-a:le-menu-souvre-au-survol into 9a07f4b on zestedesavoir:dev.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 7, 2017

Coverage Status

Coverage remained the same at 88.878% when pulling 77be3a7 on motet-a:le-menu-souvre-au-survol into 9b94ae3 on zestedesavoir:dev.

coveralls commented Aug 7, 2017

Coverage Status

Coverage remained the same at 88.878% when pulling 77be3a7 on motet-a:le-menu-souvre-au-survol into 9b94ae3 on zestedesavoir:dev.

@pierre-24

This comment has been minimized.

Show comment
Hide comment
@pierre-24

pierre-24 Aug 7, 2017

Member

Enfin <3

Member

pierre-24 commented Aug 7, 2017

Enfin <3

@pierre-24 pierre-24 merged commit a43d64b into zestedesavoir:dev Aug 7, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@pierre-24 pierre-24 added this to the Version de développement milestone Aug 7, 2017

sandhose added a commit that referenced this pull request Sep 18, 2017

Réécrit le JS des dropdowns (#4454)
* Réécrit le JS des dropdowns

Le menu principal s’ouvre au survol quand l’option est activée.

J’ai essayé d’améliorer la navigation au clavier en permettant de
basculer entre les menus avec les flèches de gauche et de droite.  La
navigation avec <tab> reste identique.

Désolé, ce commit n’est pas très atomique. Notez que j’ai retiré
l’engrenage apparaissant en haut à droite pour les utilisateurs
simple, qui faisait un peu doublon avec l’image de profil. Je pense
que de nos jour, la plupart des gens iront naturellement chercher les
réglages de leurs compte en cliquant sur leur avatar en haut à
droite : Google, GitHub et pleins d’autres sites fonctionnent comme ça.

Sur mobile, cette image de profil en haut à droite disparaît, mais
toutes les options du menu apparaissent dans le menu “hamburger” de
gauche. Il n’y a donc pas de fonctionnalités retirées.

La barre de recherche en haut à droite n’est du coup plus toujours
alignée avec les boutons du dessus. Je pense qu’il faudrait mettre une
barre de recherche qui s’agrandit quand elle prends le focus, c’est
facile à faire mais ça sera une autre PR.

Fix #661

* update.md: Désactive le mode survol pour tous les utilisateurs

* Désactive l'option pour tout les utilisateurs futurs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment