Project

General

Profile

Anomalie #4715

Pas de scroll vers le groupe de mots depuis la colonne navigation

Added by nicod _ 7 days ago. Updated 5 days ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
Start date:
04/07/2021
Due date:
% Done:

0%

Resolution:
fixed
Navigateur:

Description

Quand on a beaucoup de groupes de mots clés, un clic sur le titre du groupe dans la colonne de gauche (navigation) ne scrolle pas vers l'ancre du groupe.

En cause, un return false; inutile :
https://git.spip.net/spip/mots/src/branch/master/prive/squelettes/navigation/mots.html#L8

Une fois supprimé, on retrouve bien le comportement attendu.
Je supprime en 3.2 et 3.3 ?

© Chez moi ça marche

History

#1 Updated by b b 6 days ago

  • Status changed from Nouveau to En cours

nicod _ a écrit :

Quand on a beaucoup de groupes de mots clés, un clic sur le titre du groupe dans la colonne de gauche (navigation) ne scrolle pas vers l'ancre du groupe.

Ça n'est pas un bug mais une feature....

En cause, un return false; inutile :

Non car le js avant ce return false masque tous les groupes de la page et n'affiche que celui correspondant au titre cliqué, donc pas besoin de scroll normalement :p

Testé à l'instant sur spip.net, et ça fonctionne très bien.

#2 Updated by nicod _ 6 days ago

Je sais TRES BIEN que c'est une feature.
Et je confirme que quand il y a BEAUCOUP (pas juste 10) de groupes de mots clés, ça ne scrolle pas vers le haut.
J'ai bien testé avant de proposer.

Tu peux tester sur une install fraiche 3.2 (mais c'est pareil en 3.3) ou j'ai mis 50 groupes de mots :
https://spiptest.lerebooteux.fr/ecrire/?exec=mots (login : demo / ecureuil)

Tu cliques sur un groupe tout en bas de la colonne de gauche, ça ne remonte pas en haut.

En virant ce return false qui casse la propagation de l'événement (et donc le lien vers l'ancre), c'est ok.

Les return false dans la gestion d'events jQuery c'est un gros piège...

#3 Updated by b b 5 days ago

Haaaa ok, j'avais pas imaginé le scroll vers le haut, my bad, gogogo alors :p

PS : rasta SORS DE CE CORPS ! ^^

#5 Updated by nicod _ 5 days ago

  • Status changed from En cours to Fermé

#6 Updated by b b 5 days ago

  • Resolution set to fixed

Je ne dis pas BRAVO môsieur, maintenant qu'on est en git il est temps de se discipliner, donc fini les reports vers master ^^

Merci comme même :p

Also available in: Atom PDF