← Retour aux issues

fusionner categories et moreCategories en 1 seul fichier

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
  • fusionner les 2 yaml en 1 seul fichier définissant toutes les catégories
  • Remplacer les boutons QuickFeatureSearch par des boutons qui activent la ou les catégories correspondantes.
    • le cas où il faut activer plusieurs catégories était plus compliqué, je suis content d'avoir réussi 🤓​
    • j'ai défini une liste QuickFeatureSearch.yaml qui contient la liste des boutons, avec les catégories à activer pour chaque bouton
  • nettoyer tout le code pour ne plus distinguer categories et moreCategories
  1. Avatar Codeberg de etienneJretienneJr

    J'ai fini de nettoyer le code, mais il y a encore au moins 2 bugs que je viens de remarquer :

    • le bouton "+" disparait quand on cherche un texte
    • le bandeau des boutons rapides ne s'agrandit plus
    • j'ai perdu le SpinningDiscBorder pendant le chargement, ça serait bien de le remettre

    Au final, ça fait beaucoup de modifs dans le code (beaucoup de renommages surtout) mais côté app, il n'y a quasiment pas de modif d'UI/UX :

    • il n'y a plus de recherche de texte dans les boutons rapides, seulement dans le menu des catégories

    @laem un point à noter : dans l'annuaire, j'ai supprimé le bandeau avec les boutons rapides, qui n'est plus nécessaire vu que toutes les catégories sont dans le menu des catégories, et qui en plus poserait problème à cause des boutons qui activent plusieurs catégories d'un coup

    image

    image

  2. Avatar Codeberg de etienneJretienneJr

    @laem c'est bon j'ai corrigé les bugs que j'avais introduits, tu peux fusionner.

  3. Avatar Codeberg de etienneJretienneJr

    @laem je merge (aussi) pour pouvoir l'utiliser dans une PR à venir


✏️ Participer à la discussion