Comment reproduire le problème :
Je pense que le problème est causé par ce code.
Pouvez-vous reproduire ce problème ici sur meta ou sur try.discourse.org ?
Je peux le reproduire sur Feverbee, mais pas ici. Cela pourrait être dû aux personnalisations qu’ils ont mises en place si nous ne pouvons pas le reproduire ici.
Vous pouvez le reproduire ici :
(réduisez la largeur, mais pas trop)
Ceci est en effet un bug… cela se produit s’il n’y a que deux messages dans le sujet et que vous réduisez l’écran.
Je suis d’accord, le code lié est probablement le coupable.
Le lien devrait rester stable, car il fait partie des données initiales.
Pourrais-tu jeter un coup d’œil à celui-ci @j.jaffeux ?
Oui, c’est bien ce code qui en est responsable, et cela se produit à un certain ratio spécifique entre la hauteur de la fenêtre et celle du sujet.
Mais ce que je ne parviens pas à comprendre, c’est quel est le comportement attendu ? Je peux simplement supprimer cette vérification et cela fonctionnera, mais je suppose que nous l’avons ajoutée pour une raison ?
@eviltrout vous avez créé cette fonctionnalité initiale FIX: If a topic is really long, show the timeline · discourse/discourse@79763a6 · GitHub
Est-elle toujours valable ? La formule est-elle erronée ? Je pense qu’elle se déclenche ici alors que le sujet n’est pas vraiment si énorme, donc peut-être devrions-nous corriger cela ?
J’ai également découvert que tous les sujets suggérés étaient utilisés dans le calcul de la hauteur de ce sujet ${"#topic").css("background", "red") :
Peut-être est-ce ce qui a changé depuis que vous l’avez créée.
Peut-être devrions-nous aussi le corriger plus tôt ? Pourquoi afficher la progression du sujet dans ce cas ?
Si c’est <= 3 messages, on masque la progression du sujet si posts.height < window.height * 1.5 ?
À l’origine, la logique visait à éviter d’afficher un widget de progression si le sujet est vraiment court, ce qui me semble toujours pertinent.
Cependant, j’avais l’impression que le bug consistait en ce que le widget s’affichait, mais que le clic ne fonctionnait pas. Une solution appropriée consisterait à rendre le clic fonctionnel chaque fois que le widget est affiché. Je suis d’accord pour le garder masqué à d’autres moments (pour les sujets courts).
Je me demande si nous ne devrions pas simplement modifier le seuil de masquage à 1, au moins sur mobile / en mode réduit (voire supprimer cette logique sur mobile / en mode réduit).
Sur mobile, même si vous n’avez que deux publications, cela aide beaucoup à savoir si vous êtes sur la première ou la deuxième.
Il y a un intérêt à afficher toujours la chronologie, même sur l’ordinateur de bureau, par exemple si le message que vous lisez ne tient pas sur un écran.
Cela devrait régler le problème :
Détails dans le commit.
Ce sujet a été automatiquement fermé 5 jours après la dernière réponse. De nouvelles réponses ne sont plus autorisées.
