Employeur : ajouter un éditeur de texte markdown sur les grands champs de saisie #4834
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
EwenKorr
changed the title
EwenKorr
force-pushed
the
ewen/markdown
branch
2 times, most recently
from
082eed3 to
b46b733
Compare
EwenKorr
added
1-recette-jetable
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: pypi/pytest-randomly@3.15.0, pypi/pytest-subtests@0.13.1, pypi/pytest-xdist@3.6.1, pypi/python-crontab@3.2.0, pypi/regex@2024.9.11 |
EwenKorr
force-pushed
the
ewen/markdown
branch
2 times, most recently
from
1efa213 to
066e757
Compare
EwenKorr
commented
Oct 1, 2024
| from dotenv import load_dotenv | ||
|
|
||
| from itou.utils.enums import ItouEnvironment | ||
| from itou.utils.urls import markdown_url_set_protocol, markdown_url_set_target_blank |
Contributor
Author
There was a problem hiding this comment.
Ajouté ces callbacks pour mieux gérer les liens en markdown
EwenKorr
commented
Oct 1, 2024
Comment on lines
+560
to
+568
| MARKDOWNIFY = { | ||
| "default": { | ||
| "WHITELIST_TAGS": ["a", "p", "ul", "ol", "li", "em", "strong", "br"], | ||
| "MARKDOWN_EXTENSIONS": ["nl2br", "sane_lists"], | ||
| "LINKIFY_TEXT": { | ||
| "PARSE_URLS": True, | ||
| "CALLBACKS": [markdown_url_set_target_blank, markdown_url_set_protocol], | ||
| "PARSE_EMAIL": True, | ||
| }, | ||
| } | ||
| } |
Contributor
Author
There was a problem hiding this comment.
Nouvelle config de markdownify
Comment on lines
+5
to
+6
| const easyMDE = new EasyMDE({ | ||
| element: textarea, |
Contributor
Author
There was a problem hiding this comment.
Quelques modifs sur ce fichier : les classes principalement, pour avoir des icônes plus grandes
Comment on lines
+126
to
+137
| def markdown_url_set_target_blank(attrs, new=False): | ||
| attrs[(None, "target")] = "_blank" | ||
| attrs[(None, "rel")] = "noopener" | ||
| attrs[(None, "aria-label")] = "Ouverture dans un nouvel onglet" | ||
| return attrs | ||
|
|
||
|
|
||
| def markdown_url_set_protocol(attrs, new=False): | ||
| if href := attrs.get((None, "href")): | ||
| if not (href.startswith("http") or href.startswith("mailto")): | ||
| attrs[(None, "href")] = "https://" + href | ||
| return attrs |
Contributor
Author
There was a problem hiding this comment.
Nouveau :
- pour ouvrir les liens dans un autre onglet
- pour ajouter https:// aux liens qui n'auraient pas de protocole
EwenKorr
force-pushed
the
ewen/markdown
branch
2 times, most recently
from
39e9d87 to
d559c46
Compare
We changed the following fields: - companies: description, provided_support - job descriptions: description, profile_description - prescribers: description Two packages were added: - `django-easymde`: add the JS and CSS needed in the forms) note: as of version 1.0.4, the plugin can only have static configuration, so we cannot properly configure it in `base.py` Hence, a (temporary?) `easymde_config.js` has to be loaded. https://github.com/WPI-LNL/django-easymde/blob/main/README.md#easymde-options - `django-markdownify`: render only a chosen subset of HTML tags
EwenKorr
force-pushed
the
ewen/markdown
branch
from
d559c46 to
b19244e
Compare
tonial
approved these changes
Oct 3, 2024
xavfernandez
approved these changes
Oct 3, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR initiale : #4816
PR de revert : #4833
🤔 Pourquoi ?
Inciter les utilisateurs à renseigner plus d’infos dans leurs descriptions de fiche structure et fiche de poste.
On a observé de très bons résultats sur le marché depuis qu’ils l’ont fait.
🍰 Comment ?
utiliser
django-easymde, qui importeEasyMDE(js et css) dans les formulaires, via{{ form.media }}.Malheureusement la version actuelle du
django-easymdene permet pas de configuration "dynamique" (mentionner une fonction JS, commetoggleBoldpar exemple), donc on doit configurer l'éditeur avec un fichier JS (itou/static/js/easymde_config.js) pour le moment. On enregistre en base de données un texte au format markdown.django-markdownify: pour convertir le markdown enregistré en HTML. On préfèremarkdownifyàmarkdownici pour pouvoir choisir un sous-ensemble de balises HTML dans le rendu (<em>,<bold>,<a>…).Les tests vérifient qu'on traduit les balises désirées, et qu'on ne traduit pas les autres balises.
On ne teste pas le bon chargement de l'éditeur EasyMDE (ça ferait partie de tests fronts ?).
Nouveautés avec cette nouvelle PR :
br) en HTML.[test](www.test.com)=> on rajoutehttps://)🚨 À vérifier
🏝️ Comment tester + captures d'écran
En tant qu'employeur
Se connecter en tant qu'employeur et créer une nouvelle structure
Aller voir la fiche publique
Modifier les informations
Prévisualiser
En tant que prescripteur
Se connecter en tant qu'Orienteur (prescripteur non habilité) et "Modifier cette organisation"
Cas particulier pour France Travail (se connecter en tant que Prescripteur habilité), les champs sont en lecture seule :
