← Retour aux issues

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.

Avatar Codeberg de etienneJretienneJr

ç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é category en group dans categories.yaml et moreCategories.yaml (attention aux conflits avec les PR en cours sur les icones !) et aussi dans useFetchDrawBikeParkings.ts (j'ai un petit doute pour ce dernier, à reviewer svp)
  • j'ai corrigé tous les scripts qui utilisaient categoryColors.yaml ou qui listaient les catégories en lisant leur propriété .category
  1. Avatar Codeberg de GhostGhost

    Dokploy Preview Deployment

    Name Status Preview Updated (UTC)
    web-master ❌ Failed Preview URL 2025-03-10T08:05:03.979Z
  2. Avatar Codeberg de etienneJretienneJr

    @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 ?

  3. Avatar Codeberg de laemlaem

    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.

  4. Avatar Codeberg de etienneJretienneJr

    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".

  5. Avatar Codeberg de etienneJretienneJr

    J'ai rebasé ce que j'avais fait au début de cette PR, pour utiliser le mot group pour les groupes de catégories, en particulier dans moreCategories.yaml. En complément j'ai aussi :

    • rajouté une propriété hidden facultative 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 @media qui 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 ?

  6. Avatar Codeberg de laemlaem

    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:


✏️ Participer à la discussion