uniformisation de l'usage de group pour les groupes de catégories + gestion des catégories cachées
publié le , mis à jour⚙️ Ceci est une proposition de changement de code.
Rendez-vous sur sa page Coderberg pour l'osculter.
ça me perturbait trop que les catégories aient une propriété qui s'appelle elle aussi category pour en fait désigner le groupe de catégories auquel elles sont rattachées. D'ailleurs le mot group était déjà utilisé à quelques endroits, donc j'ai uniformisé pour l'utiliser partout :
- j'ai renommé le yaml des couleurs en
categoryGroupColors.yaml - j'ai renommé la propriété
categoryengroupdanscategories.yamletmoreCategories.yaml(attention aux conflits avec les PR en cours sur les icones !) et aussi dansuseFetchDrawBikeParkings.ts(j'ai un petit doute pour ce dernier, à reviewer svp) - j'ai corrigé tous les scripts qui utilisaient
categoryColors.yamlou qui listaient les catégories en lisant leur propriété.category
GhostDokploy Preview Deployment
Name Status Preview Updated (UTC) web-master ❌ Failed Preview URL 2025-03-10T08:05:03.979Z etienneJr
@mael pour l'instant les groupes sont créés en lisant moreCategories.yaml. Est ce que ça ne serait pas plus simple qu'il soient définis dans le yaml qui définit pour l'instant uniquement les couleurs ?
laem
Est ce que ça ne serait pas plus simple qu'il soient définis dans le yaml qui définit pour l'instant uniquement les couleurs ?
Oui, mais je ne suis pas sur que ce soit un changement important par rapport au fait d'ajouter plus de catégories par exemple. C'est un changement interne seulement.
etienneJr
Oui, mais je ne suis pas sur que ce soit un changement important par rapport au fait d'ajouter plus de catégories par exemple. C'est un changement interne seulement.
Pas uniquement, je me dis que ça permettrait des définir un groupe comme invisible, pour qu'il n'apparaisse pas dans le menu des catégories, mais qu'on puisse définir des catégories cachées dans ce groupe. Par exemple un groupe "catégories non classées" ou un groupe "vérification qualité des tags".
etienneJr
J'ai rebasé ce que j'avais fait au début de cette PR, pour utiliser le mot
grouppour les groupes de catégories, en particulier dansmoreCategories.yaml. En complément j'ai aussi :- rajouté une propriété
hiddenfacultative qui va permettre de créer des catégories qui n'apparaitront que via la barre de recherche - géré le non affichage de ces catégories cachées dans le menu (sauf si on est en train de faire une recherche ou si la catégorie est active)
- pour l'exemple j'ai caché les sports pour ne laisser afficher sur les terrains de sport
- corrigé un bug qui faisait que, en mobile, le bouton pour étendre un groupe apparaissait lors de la recherche de texte
- mis à jour le message d'astuce
- corrigé un bug lié à l'utilisation des
@mediaqui faisait que dans certains cas (écran large avec fonction tactile) le bouton pour étendre la catégorie n'apparaissait pas.
Edit : le preview dispo ci dessus fonctionne bien.
@laem il me reste un dernier hic que je n'ai pas réussi à corriger : j'ai remarqué que le background du volet n'est pas le même en mobile (il est blanc) et en desktop (il est bleu clair). Ca fait que dans le fameux cas d'un écran large avec fonction tactile, le dégradé sous le ExpandButton n'est plus adapté (il est prévu pour le fond blanc du cas mobile, alors que le fond est alors bleu clair). J'ai cherché où est défini le fond blanc du cas mobile (du moins du cas écran de moins de 600px) pour le changer en bleu clair, mais je n'ai pas trouvé où c'est... Si tu m'indiques où c'est je pourrai corriger. On met les 2 en bleu clair, n'est-ce pas ?
- rajouté une propriété
laem
Trop cool, merci pour le travail. J'ai corrigé le pb de fond de couleur, bien vu ! Un passage au blanc n'est pas exclut, à voir plus tard :+1: