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

Corrige une marge sur une liste de tutos/articles de la bibliothèque #6133

Merged
merged 3 commits into from Sep 11, 2021

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Jun 7, 2021

Fix #6109.

Cette PR corrige un petit bug graphique dans une liste de tuto/articles de la bilbiothèque quand on liste sur un type de contenu dans une sous-catégorie.

Contrôle qualité

  • s'arranger pour avoir une page de tutos dans la même sous-catégorie sans pagination (possible de jouer avec le paramètre dans le backend content_per_page, le type d'URL concerné est le suivant : http://localhost:8000/bibliotheque/?subcategory=00-etude&type=tutorial ;
  • vérifier que l'affichage est bon (pas de vignette collée en bas) ;
  • s'arranger pour avoir de la pagination ;
  • vérifier que l'affichage est identique à l'actuel (pas de marges supplémentaires introduite par cette PR).

@Arnaud-D Arnaud-D added the C-Front Concerne l'interface du site label Jun 7, 2021
@Arnaud-D Arnaud-D added this to En développement in Suivi des PR via automation Jun 7, 2021
@Arnaud-D Arnaud-D moved this from En développement to En attente de QA in Suivi des PR Jun 7, 2021
@coveralls
Copy link

coveralls commented Jun 7, 2021

Coverage Status

Coverage remained the same at 86.621% when pulling 3168343 on Arnaud-D:dev into 7608ec8 on zestedesavoir:dev.

@philippemilink
Copy link
Member

Rapport de QA

Pas OK ❌

C'est bien corrigé lorsqu'il n'y a pas de pagination.

Avec pagination, la marge entre la pagination en haut et le contenu a disparu (mais la marge recherchée est correcte en bas):
image

@philippemilink philippemilink moved this from En attente de QA to Modification demandée in Suivi des PR Aug 20, 2021
@Arnaud-D
Copy link
Contributor Author

Arnaud-D commented Sep 2, 2021

Je crois que c'est bon désormais.

@Arnaud-D Arnaud-D moved this from Modification demandée to En attente de QA in Suivi des PR Sep 2, 2021
Suivi des PR automation moved this from En attente de QA to Fusionnable après rebase Sep 11, 2021
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.

Rapport de QA

OK ✔️

@philippemilink philippemilink enabled auto-merge (squash) September 11, 2021 16:25
@philippemilink philippemilink merged commit 91b28d6 into zestedesavoir:dev Sep 11, 2021
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Sep 11, 2021
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
Suivi des PR
  
Fusionnée
Development

Successfully merging this pull request may close these issues.

Espace manquant en bas de la liste de contenu
3 participants