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

Utilise la police monospace pour le nouvel éditeur peu importe sa localisation #6423

Merged
merged 2 commits into from
Dec 5, 2022

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Nov 10, 2022

Actuellement, le nouvel éditeur utilise la police monospace seulement lorsqu'il est dans un .message-content ou .article-content, ce qui n'est pas tout le temps le cas. Cette PR impose l'utilisation de la police monospace tout le temps pour le nouvel éditeur.

Fixes #6421

QA : Vérifier que l'éditeur utilise bien la police monospace sur les pages de contenus (réponse sur un sujet, commentaire d'un contenu) mais aussi sur les autres (édition d'un message, création d'un sujet, etc.)

@Situphen Situphen added S-BUG Corrige un problème C-Front Concerne l'interface du site labels Nov 10, 2022
@coveralls
Copy link

coveralls commented Nov 10, 2022

Coverage Status

Coverage remained the same at 88.299% when pulling ba03d2b on Situphen:monospace into 9940a37 on zestedesavoir:dev.

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

Testé en voulant commenter un article (c'est bien une police monospace) et en créant un sujet sur le forum (ce n'était pas une police monospace, maintenant ça l'est).

QA OK ✔️

@philippemilink philippemilink enabled auto-merge (squash) December 5, 2022 21:07
@philippemilink philippemilink merged commit a333b4c into zestedesavoir:dev Dec 5, 2022
@Situphen Situphen deleted the monospace branch December 5, 2022 21:22
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 S-BUG Corrige un problème
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Deux problèmes avec le nouvel éditeur
3 participants