style(Homeworks): update text color for completed items

[Cleboost]

🚀 Nouvelle Pull Request

Proposez vos modifications pour améliorer Papillon

Informations importantes

Merci de vous référer à la documentation sur la contribution si vous avez des questions à propos des pull requests (https://gitbook.getpapillon.xyz/organisation/outils-internes/github)

Checklist d'avant pull request

Veuillez cocher toutes les cases applicables en remplaçant [ ] par [x].

• Vous avez testé de build le projet avec vos modifications et ce build a réussi
• Vous respectez les conventions de codage et de nommage du projet
• Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
• [?] Cette pull request n'est pas un duplicata d'une autre
• Cette pull request est prête à être revue (review) et fusionnée (merge)
• Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
• Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
• Les détails des changements ont été décrits ci-dessous
• Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

C'est un duplicata de mon ancienne PR : #184
Mais qui s'est perdu au milieu des updates de Vincent.

Informations supplémentaires

[Cleboost]

Juste je ne comprends pas pourquoi Eslint refuse mon code en citant des fichiers que je n'ai pas modifiés. Je pense que mon code n'a aucun impact sur les autres fichiers.

[Louis454545]

En vrai ça serait bien de limiter le devoir a 1 seule ligne, comme une pr l'avais fait mais JSP laquelle

[Cleboost]

Alors ce que j'avais fais c'est que une fois qui est coché il passe en une ligne.
Un autre gars avait gérer le menu. Je refait le passage sur une ligne dcp ?

[Louis454545]

Alors ce que j'avais fais c'est que une fois qui est coché il passe en une ligne. Un autre gars avait gérer le menu. Je refait le passage sur une ligne dcp ?

ouais pour moi c'est plsu comprehensible, et aussi peut etre rayer le devoir comme un todo list

[Kgeek33]

Perso, même si je pense que c'est une bonne idée, cette fonctionnalité peut ne servir à rien (retour à la ligne), car quand un prof fait juste un retour à la ligne dans son devoir, ben la limite de ligne ne fonctionne plus. Exemple :

[Kgeek33]

ok non eft, c'est la nouvelle version de TypeScript qui est comme ça
Sera corrigé dans la pr #370 dc

[Kgeek33]

Sinon, LGTM 👍

[Louis454545]

Perso, même si je pense que c'est une bonne idée, cette fonctionnalité peut ne servir à rien (retour à la ligne), car quand un prof fait juste un retour à la ligne dans son devoir, ben la limite de ligne ne fonctionne plus. Exemple :

Pas compris, ya pas rapport non ?

[Kgeek33]

Prenons l'exemple de ma capture. Si on limite à 2 lignes par exemple ;
Comportement attendu :

Préparer le devoir surveillé n°3 sur le chapitre 3 (vous devez aussi maîtriser le...

Comportement au temps où c'était intégrer la limite (2 lignes) :

Préparer le devoir surveillé n°3 sur le chapitre 3 (vous devez aussi maîtriser le...
Ne pas oublier la calculatrice. Lors d'une évaluation, les échanges de matériel entre...
Vous pouvez préparer une copie double avec NOM, Prénom, Classe, Date, Devoir...
Le DS3 aura lieu de 10h à 11h30 en salle des devoirs sous la surveillance de collègues de...

C'est pour ça que je dis que ça sert à rien d'intégrer une limite pour que ça s'affiche comme ça

[Cleboost]

Dcp je changes un truc ou pas ?

[Kgeek33]

Non laisse comme c'est, perso c'est pas dérangeant qu'il n'y ait pas de limite
Comme ça, ça évite que l'utilisateur clique sur le devoir pour voir le devoir complet (en plus, il y a déjà la limite sur l'accueil)

[Louis454545]

Raye le aussi

[ecnivtwelve]

LGTM