Flash Chat et la date - Page 1

Ce n'est probablement pas possible avec l'infrastructure actuelle, mais un truc qui manque vraiment au flash chat c'est l'info sur le jour ou un message a été posté.


Une solution qui permetrais de ne pas le changer fondamentalement ca pourrais etre d'afficher entre deux messages une ligne qui indique le jour un peu comme le fait discord:





TL;DR: Si un message est posté et qui n'est pas du même jour que le dernier message, une ligne entre les deux messages indique le date/jour ou le dernier message à été posté.

et mettre tout simplement un tooltip sur l'heure pour afficher la date ?

Personellement moins pratique et pas tres mobile compatible

!patch

Edit: hmmmm pas sur de comment deval marche et si c'est possible de ne le faire que a la generation de la page (et non un truc a ajouter a la BDD) je pense que le faire a la génération serait mieux (car demande moins de changements)

L'idée est bonne et je suis d'accord : c'est de la mise en forme donc mieux vaut faire ça au rendu sans changer les données.

Pour Deval j'ai essayé d'écrire une doc mais c'est pas forcément très complet. Il doit y avoir moyen de s'en sortir en groupant les messages par jour puis en utilisant 2 boucles : une pour parcourir les jours puis une seconde pour parcourir les messages d'un jour. Quelque chose dans ce genre-là, modulo les erreurs :

{{ for day, shouts in group(shouts, (s) => tra("nom de la locale pour formatter le jour", shout.time), (s) => [s], (a, b) => cat(a, b)) }} 
  <span class="day">{{ $ day }}</span>
  {{ for shout in shouts }}
    /* Comme avant */
  {{ end }}
{{ end }} 

Je regarde prochainement sauf si qqun s'y met entre temps

Je vais regarder et merci pour l'idée et template (je regardais la doc de Deval mais entre la fatigue et autre pas vraiment de grande idée de comment procéder xD, c'est d'ailleurs pour ca le pull request pour corriger une typo dans la doc )

J'ai un truc qui semble marcher

Ca ne fait pas exactement ce que je voulais parce qu'il semble que tu utilise date pour la conversion des dates et non strftime donc on a droit qu'au nom de jour et mois en anglais; donc je n'utilise que des chiffres

Bref je pousse les changements sur ma fourchette


Edit:

Ca donne un rendu de ce type:

La ligne de date est bien sur avant les posts de la dite date (vu que le FC affiche du plus recent au plus vieux)



Edit2: Pour avoir faire des tests je te proposerais bien des changements pour pouvoir un format texte pour le texte (sans casser l'existant) mais ca ne garantit pas que ca marche, il faut faire un changement dans losp pour pouvoir appeler strftime, et il faut aussi que "setlocale" soit appelé quelque part.

Problème: si la locale n'existe pas sur le serveur, ca foire de manière sympathique: ca retourne une chaine vide plutôt qu'un fallback vers l'anglais o_o

Bien entendu la CSS est faite a mon gout tu peux prendre le tout tel quel, ou changer dans le style que tu pense collerais mieux a yN.

Faut encadrer la date avec des ↑ pour rappeler que c'est ce qui précède qui est à cette date ^^

-------↑ Aujourd'hui ↑------

les gens

Oh tu m'as pris de vitesse, ça a l'air très bien

Je regarde ce soir, merci !

De rien, tu avais fait le plus gros du boulot avec le bout de code deval. A vrai dire je n'aurai jamais pu le faire sans.
Le plus long a été de corriger d'en comprendre le fonctionnement et de corriger le bug

J'ai repris ton commit pour faire des renommages mineurs de qq variables et en profiter pour mettre les nouveaux styles CSS dans un fichier qui pourra être partagé avec d'autres pages si besoin. Visuellement rien n'a changé, je te laisse me dire si ça te convient toujours avant de merger

Pas grand chose à dire, c’est la même chose avec juste des changement “cosmétiques”

Oui la modification reste identique c'est seulement de la forme, je voulais être sûr de ne pas avoir perdu quelque chose en route. C'est mergé et déployé, merci pour le boulot

Et c'était la toute première contribution tierce à yAronet

Ah oui, c'est carrément plus clair

Histoire de chipoter, je soutiens la suggestion de Nil (post ./9), ça apporterait un peu de clarté supplémentaire

Merci à tous les deux, c'est vrai que ça change pas mal de choses au niveau suivi ! (et je soutiens aussi ma proposition )

Bravo bravo !!

Manque plus que les smileys standards dans le FC, ainsi que le FC dans la page du Suivi Sujets

Je te laisse la charge de ce projet

C'est fait J'en ai profité pour changer le format et afficher le nom du mois en entier puisqu'il y avait la place.

Superbe, c'est devenu parfait, merci à vous !