Je pense que la chronologie pourrait être masquée sur les sujets sans aucune réponse, ou que la hauteur minimale du conteneur des messages devrait être à peu près la même que celle de la chronologie ?
Si ma mémoire est bonne, cela existe depuis un moment et est inhérent aux petits écrans et aux sujets d’une seule ligne, mais peut-être avons-nous régressé ?
@awesomerobot m’a rappelé qu’on avait l’habitude de masquer la timeline pour les sujets à un seul message, je crois ? Peut-être que c’est pour cela qu’on le remarque davantage, ou peut-être s’agit-il d’une régression réelle ? Dans ce cas, tous les éléments de la timeline sont en effet dupliqués dans les contrôles en bas du premier (et unique) message…
Le gros problème avec les publications uniques, c’est que lorsqu’elles sont réellement utiles comme celle-ci, elles ne fonctionnent pas correctement. Nous ne prenons pas en charge la navigation à mi-parcours d’une publication géante, ce qui devient extrêmement, extrêmement, extrêmement, extrêmement confus. Bien pire que le simple problème de rendu.
Fais-le en fonction de la longueur du message plus le message unique alors… nous n’avons pas besoin d’une chronologie verticale pour un message unique d’une seule phrase dans un seul sujet, n’est-ce pas ?
Absolument pas, mais nous devons corriger l’implémentation, je pense que c’est ce que j’essaie de dire.
Si je devais choisir entre le supprimer sans condition ou le conserver pour des articles uniques très longs et atypiques, je choisirais de le supprimer sans condition.
Cependant, si nous corrigeons l’implémentation sous-jacente, je soutiendrais tout à fait de le conserver pour le guide incroyable de @Johani, ce serait super utile.
si un seul message dans le sujet ET si ce seul message contient moins de .. 1000* caractères .. alors masquer complètement la chronologie
Même deux messages d’une seule phrase devraient probablement être sûrs, non ?
mais que se passe-t-il s’ils ne contiennent que des caractères uniques et des retours chariot/saut de ligne, etc. ? Je ne sais pas. Je pense qu’une vérification approximative suffit ; il n’est pas nécessaire de se soucier de cas limites extrêmes ici.
Bien sûr, on peut faire quelque chose ici, même en utilisant uniquement du CSS. Mais il faudra corriger la barre de défilement si on opte pour cette approche.
Sinon, on pourrait tout aussi bien simplement passer à if 1 post then hide timeline, ce qui est encore plus simple.
Si nous laissons cela tel quel pour les sujets avec un seul énorme message, cela ne fonctionne tout simplement pas correctement, ce qui est très déroutant. Nous devons donc corriger cela.
Voici ma tentative de correction du chevauchement de la chronologie :
Cela ne résout pas ce que vous avez expliqué, à savoir que cliquer ou glisser sur la chronologie ne modifie pas la position de défilement de la fenêtre. J’ai jeté un coup d’œil rapide à la mise en œuvre de cela, et je pense que cela devrait faire l’objet d’un projet dédié : cela a toujours été ainsi et le changement aurait de nombreuses implications.
Si nous n’avons qu’un seul message, je vérifierai maintenant sa hauteur et masquerai le défileur si elle est inférieure à 1000 px.
Cela représente un message d’environ cette longueur :
Ce commit corrige également un autre bug qui faisait apparaître le bouton d’administration du sujet à une position incorrecte lors d’un rafraîchissement complet de la page :