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

Modification de django-prose pour permettre la sélection des entêtes h2 à h6 et p #1451

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

alemangui
Copy link
Collaborator

@alemangui alemangui commented Jan 10, 2025

Closes #1450

Contexte

django-prose (et Trix) n'ont que l'entête h1 par défaut dans l'éditeur. Ceci pose un problème pour l'accessibilité et SEO. Pour info j'ai levé un issue ici.

Solution

J'ai créé un nouveau field EnrichedRichTextField qui hérite de celui de django-prose. Le widget EnrichedRichTextField a été ajouté également et hérite aussi du widget django-prose. Le seule différence est que le widget charge aussi "extend-buttons.js", un fichier javascript utilisé pour modifier le comportement de l'éditeur Trix.

extend-buttons.js

Ce fichier ajoute un listener sur l'event trix-before-initialize, lancé par Trix, pour programmatiquement modifier la barre de boutons de l'éditeur. Notamment :

  • Enlève le bouton Titre h1, nous ne le voulons pas car le titre de la page a déjà l'entête h1.
  • Ajoute le bouton P pour le paragraphe
  • Ajoute les boutons h2 à h6 pour les différentes entêtes

Avant

image

Après

image

image

@alemangui alemangui requested a review from pletelli January 10, 2025 16:22
Copy link
Collaborator

@pletelli pletelli left a comment

Choose a reason for hiding this comment

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

Waw c'est top de pouvoir modifier la toolbar comme ça !

@alemangui alemangui merged commit af95e49 into staging Jan 13, 2025
5 checks passed
@alemangui alemangui deleted the 1450-prose-headings branch January 13, 2025 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ajouter le support pour les différents entêtes html dans prose-editor
2 participants