Vendor la dépendance easyMDE

[francoisfreitag]

🤔 Pourquoi ?

Éviter le problème de dependabot avec setuptools (exemples: #5181-#5195).
Parce que la lib d’intégration a pas mal de défaut et semble non maintenue.
Pour mieux indiquer nos dépendances JS (plutôt que de passer une lib qui vendor pour nous).

🚨 À vérifier

Plutôt que de passer par un form Media qui est peu clair, j’ai ajouté la dépendance directement dans base.html. Tout le monde devra le charger une fois, puis on n’en parlera plus.

Comment tester ?

Valider les éditeurs de texte riche sur les pages de :

• description fiche de poste employeur
• description entreprise et accompagnement employeur
• description orga prescripteur

💻 Captures d'écran

[EwenKorr]

Peut-être des {% blocks script %} et {% block extra_head %} ?

Je pense à :

• itou/templates/companies/create_siae.html
• itou/templates/companies/edit_job_description_details.html
• itou/templates/companies/edit_siae_description.html
• itou/templates/prescribers/edit_organization.html

(introduits dans #4834)

[francoisfreitag]

Je les ai mis dans base.html pour ne pas avoir à gérer du JS par page. Les gérer par page est également possible. Pour le JS qui est utile sur plusieurs pages, j’ai tendance à le rendre disponible partout pour éviter la surprise que le script ne soit pas disponible. 🤷

[EwenKorr]

Mince, j'ai oublié un mot dans mon commentaire 🤦

Peut-être supprimer les blocs dans les fichiers que j'ai listés ?
Je les avais ajoutés page par page justement, et s'ils sont dans base.html, ces blocs sont inutiles dans le template, j'imagine.

[francoisfreitag]

Ahh, avec le mot en plus je comprends mieux ! Très bonne remarque, c’est incorporé.

[github-actions]

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

[francoisfreitag]

Est-ce qu’on ne pourrait pas se passer de fontawesome (en utilisant la config appropriée ?).
J’investiguerai vendredi.

[xavfernandez]

Est-ce qu’on ne pourrait pas se passer de fontawesome (en utilisant la config appropriée ?). J’investiguerai vendredi.

👍 Ça me semblerait effectivement mieux que la CSP un peu moche 😬
A priori mettre autoDownloadFontAwesome à false (https://github.com/gip-inclusion/les-emplois/pull/5196/files#diff-20f58c734e7eca8ec6240b9a15fa3ee29518b99bd6dbb377eb2062bbbc058ec3R7 ) devrait suffire (en incluant bien font-awesome.min.css dans nos fichiers statics)

[EwenKorr]

D'après

les-emplois/itou/static/js/easymde_config.js

Line 12 in 95d9fc0

className: "ri ri-bold ri-lg",

c'est RemixIcon qu'on utilise, on n'aurait pas besoin de télécharger FontAwesome a priori. La version téléchargée est vieille en plus. Je ne me rappelle plus pourquoi j'ai laissé autoDownloadFontAwesome à true 🤔
Sans doute un oubli après trop de ping-pong dans la petite discussion UI : https://www.notion.so/plateforme-inclusion/L-icone-liste-me-fait-penser-l-option-d-alignement-du-texte-10ee8fa5c35b8083b2e3c218c1f670fe

[francoisfreitag]

Je confirme que ça marche bien avec autoDownloadFontAwesome: false :)