Repository: zestedesavoir/zds-site

Issue:
Navigation au survol
La navigation dans les menus au survol ne fonctionne pas, même une fois l'option activée dans le profil.



Discussion:
PS: on ne peut plus mettre de labels nous-même sur les issues ?

> La navigation dans les menus au survol ne fonctionne pas, même une fois l'option activée dans le profil.

Sur la version actuellement sur la branche dev (équivalent de la prod et non la préprod), je navigue en ce moment en survol.

> PS: on ne peut plus mettre de labels nous-même sur les issues ?

La gestion du dépot principal est réservé à un ensemble restreint de personnes (plus efficaces pour la gestion des issues/pr/merges)

> Sur la version actuellement sur la branche dev (équivalent de la prod et non la préprod), je navigue en ce moment en survol.

L'issue était pour Alex, le bug est présent sur la preprod (donc la branche intégration si j'ai bien compris).

> L'issue était pour Alex

Ce n'était pas précisé dans ton message. Mais merci du coup pour le rapport

Le menu au survol est une chose relativement "complexe" à gérer proprement (ie : ne pas ouvrir les box à l'arrache, etc).

Je repousse ça à pendant/après la beta. Faudrait du coup que je masque l'option peut-être ?

Éventuellement oui, que ça ne soit pas considéré comme un bug oublié
pendant la beta.
Mais n'en profite pas pour oublier, j'y tiens à cette navigation moi :p (ce
n'est pas la première fois que je te fais chier avec il me semble..^^)

Le 1 juin 2014 23:39, Alexandre Demode notifications@github.com a écrit :

> Le menu au survol est une chose relativement "complexe" à gérer proprement
> (ie : ne pas ouvrir les box à l'arrache, etc).
> 
> Je repousse ça à pendant/après la beta. Faudrait du coup que je masque
> l'option peut-être ?
> 
> —
> Reply to this email directly or view it on GitHub
> https://github.com/zestedesavoir/zds-site/issues/661#issuecomment-44790196
> .

Bah c'est ici, je ne peux pas l'oublier :)

Fonctionnalité non intégré au front (Juste cette ligne : https://github.com/zestedesavoir/zds-site/blob/93abd4894706145228675ca9887e8eb230fd788f/templates/base.html#L167).

On notera le nom particulièrement ambigu du paramètre d'ailleurs : il est
vrai quand c'est hover ou quand c'est click ?

2015-02-11 19:21 GMT+01:00 A-312 notifications@github.com:

> Fonctionnalité non intégré au front (Juste cette ligne :
> https://github.com/zestedesavoir/zds-site/blob/93abd4894706145228675ca9887e8eb230fd788f/templates/base.html#L167
> ).
> 
> —
> Reply to this email directly or view it on GitHub
> https://github.com/zestedesavoir/zds-site/issues/661#issuecomment-73934059
> .

Ce bug a été remonté y'a quelques jours sur le forum.

Comme le mentionne récemment @Loodoor via son issue, c'est plus un bug qu'une évolution puisque l'option existe déjà. Je change le tag.

C'est une partie des choses que j'ai tenté de relancer via cette ZEP qui n'a pas attiré grand monde : https://zestedesavoir.com/forums/sujet/4491/zep-37-reorganisation-des-menus-deroulants-du-header/

Je remonte cette issue : les booléens python/js ne sont pas compatibles (True/true et False/false)

donc il faudrait plutôt un getter pour avoir une valeur js compréhensible

https://github.com/zestedesavoir/zds-site/blob/93abd4894706145228675ca9887e8eb230fd788f/templates/base.html#L167

sur cette ligne on aura soit data-is-hover="True" soit ="False" donc

j'ai modifié à la main le html, et en mettant ="false", ca fonctionne ! une fois sur deux, certes

Ou `truc = 'true' if machin else 'false'`

Ou `str(x).lower()`.


Resolving patch:
diff --git a/zds/member/models.py b/zds/member/models.py
--- a/zds/member/models.py
+++ b/zds/member/models.py
@@ -63,7 +63,7 @@ class Meta:
     github_token = models.TextField('GitHub', blank=True)
     show_sign = models.BooleanField('Voir les signatures', default=True)
     # do UI components open by hovering them, or is clicking on them required?
-    is_hover_enabled = models.BooleanField('Déroulement au survol ?', default=True)
+    is_hover_enabled = models.BooleanField('Déroulement au survol ?', default=False)
     allow_temp_visual_changes = models.BooleanField('Activer les changements visuels temporaires', default=True)
     show_markdown_help = models.BooleanField("Afficher l'aide Markdown dans l'éditeur", default=True)
     email_for_answer = models.BooleanField('Envoyer pour les réponse MP', default=False)


Test patch: