Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Affiche les objectifs d'une publication dans son en-tête (et corrige un souci avec l'affichage des catégories) #6440

Merged
merged 3 commits into from
Jan 15, 2023

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Dec 21, 2022

En lien avec la nouvelle classification par objectifs (voir le projet associé).

Cette PR ajoute l'affichage des objectifs d'une publication dans son en-tête, juste en dessous des catégories. J'ai repris un affichage similaire aux catégories (et même un peu plus simple). Le but premier est juste de rendre les objectifs visibles, pas de faire un bel affichage. C'est une évolution qui peut être faite après. Je suis allé au plus simple, délibérément.

Au passage, j'ai corrigé un petit bug d'affichage dans la liste des catégories (des espaces en trop dans certains cas).

Contrôle qualité

  • Se connecter en tant que staff (admin par exemple fait l'affaire)
  • Aller sur une publication
  • Jouer avec les objectifs et regarder l'affichage dans l'en-tête avec 0, 1, 2 et 3 objectifs. Vérifier que c'est propre.
  • Faire pareil avec les catégories, vu que j'ai ajusté ça aussi à la marge.

@Arnaud-D Arnaud-D added the C-Front Concerne l'interface du site label Dec 21, 2022
@coveralls
Copy link

coveralls commented Dec 21, 2022

Coverage Status

Coverage: 88.296%. Remained the same when pulling db07be7 on Arnaud-D:objectifs_dans_en-tête into 7041043 on zestedesavoir:dev.

@philippemilink
Copy link
Member

Ça semble bien fonctionner, mais pourquoi pour les catégories on affiche "Catégorie 1 et Catégorie 2", mais pour les objectifs on affiche "Objectif 1**,** Objectif 2" ? Je serais pour avoir les mêmes séparateurs pour les deux.

@Arnaud-D
Copy link
Contributor Author

Le code est plus simple avec que des virgules et ça évite de faire quelque chose de plus fragile encore (notament niveau espacement) pour un gain négligeable.

Je n'ai pas voulu changer trop les catégories, parce que ça semble marcher suffisamment tel que c'est fait, et je ne voulais rien casser. Et c'est pas le but de la PR, j'ai juste changé une bricole en passant.

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

@philippemilink philippemilink enabled auto-merge (squash) January 15, 2023 14:47
@philippemilink philippemilink merged commit 7e59f7a into zestedesavoir:dev Jan 15, 2023
@Arnaud-D Arnaud-D deleted the objectifs_dans_en-tête branch January 15, 2023 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants