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

Améliorations mineures #55

Merged
merged 9 commits into from
Nov 29, 2023
Merged

Améliorations mineures #55

merged 9 commits into from
Nov 29, 2023

Conversation

Ash-Crow
Copy link
Collaborator

@Ash-Crow Ash-Crow commented Nov 27, 2023

🎯 Objectif

  • Retirer les warnings actuellement présents quand on lance le site
  • Traiter les issues ouvertes qui peuvent l'être rapidement

🔍 Implémentation

  • Ajout d’une app dashboard dédiée à personnaliser l’admin de Wagtail
    • Ajout de la barre d’outils pour les utilisateurs connectés (en remplaçant l'icône oiseau peu claire par un stylo)
  • Remplacement de l’app obsolète wagtail.contrib.modeladmin par le paquet wagtail-modeladmin (provoquait un RemovedInWagtail60Warning)
  • Retrait du padding en haut des images dans le bloc "Image à gauche et texte à droite" (résout Spacing en trop pour le bloque image (2 colonne) #3 )
  • Mise à jour de django-dsfr
  • Ajout d’un paramètre heading_tag au block callout pour résoudre une alerte d’accessibilité
  • Ajout dans la configuration du site d’un paramètre permettant l’ajout d’un lien vers la modale de choix de thème

⚠️ Informations supplémentaires

Cartes laissées en attente d’infos supplémentaires :

À faire dans une PR dédiée :

Issues concernant la section promotionnelle :

🏕 Amélioration continue

  • Déplacement de la logique des blocs depuis models.py vers un blocks.py dédié.

🖼️ Images

(optionnel) Une ou plusieurs captures d’écran, si pertinent

@Ash-Crow Ash-Crow changed the title Améliorations mineurs Améliorations mineures Nov 27, 2023
@Ash-Crow Ash-Crow added the amélioration Nouvelle fonctionalité ou demande label Nov 27, 2023
@Ash-Crow Ash-Crow linked an issue Nov 27, 2023 that may be closed by this pull request
badge = BadgeBlock(label="Badge")


heading_choices = [

Choose a reason for hiding this comment

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

Ça m'embête un peu, ces variables qui se promènent entre deux définitions de classe. Y a-t-il moyen de les définir directement dans leurs classes ?
Un peu comme dans Django quand on a des CharField avec des choix :

class Student(models.Model):
    class YearInSchool(models.TextChoices):
        FRESHMAN = "FR", _("Freshman")
        SOPHOMORE = "SO", _("Sophomore")
        JUNIOR = "JR", _("Junior")
        SENIOR = "SR", _("Senior")
        GRADUATE = "GR", _("Graduate")

    year_in_school = models.CharField(
        max_length=2,
        choices=YearInSchool.choices,
        default=YearInSchool.FRESHMAN,
    )

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ça m'embête aussi un peu, je les ai laissées avant leur classe (comme elles étaient quand j'ai déplacé le tout depuis models.py), mais je serai effectivement d'avis de soi les intégrer dans la classe, soit les déplacer dans un constants.py

@Ash-Crow Ash-Crow merged commit c708174 into main Nov 29, 2023
1 check passed
@Ash-Crow Ash-Crow deleted the minor-improvements branch January 3, 2024 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
amélioration Nouvelle fonctionalité ou demande
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Spacing en trop pour le bloque image (2 colonne)
2 participants