search

philharmoniedeparis / metascore-library

The front-end library that powers the metaScore authoring tool.
https://metascore.philharmoniedeparis.fr
CeCILL Free Software License Agreement v2.1
0 stars 0 forks source link

Éliminer l'utilisation d'une iframe dans l'éditeur #628

Closed semiaddict closed 1 year ago

semiaddict commented 1 year ago

CKEditor ne fonctionne pas correctement lors de l'édition de contenu dans une iframe, il a été décidé de supprimer l'utilisation de l'iframe dans l'éditeur metaScore

Pour se faire, les tâches suivantes doivent être accomplies :

Cela devrait, je l'espère, résoudre les problèmes suivants : #564, #567, #597 et #626

semiaddict commented 1 year ago

Je rencontre un souci avec la CSS de certains guides (comme jR6z), qui modifient la taille de police au niveau de l'élément <html>, car cela impacte toutes les mesures en rem dans la page. rem est une unité de mesure relative qui est basée sur la taille de police de la racine (l'élément <html>). Donc si je remplace html par .metaScore-app dans la CSS, toutes les mesures en rem ne correspondront plus à ce qui est prévu par l'auteur.e.

Savez-vous pourquoi il y a des paragraphes dans les applications qui utilisent rem pour la taille de la police ? D'après une recherche rapide dans la BDD, cela semble concernait 18 applications:

lmoreau-gaudry commented 1 year ago

Avec Anne-Florence, on a regardé ça. Parmi ces exemples, seulement quatre guides (jR6z, OYgL, pYX1 et yPmW) sont publics. Le plus simple est de faire les modifications à la main des fichiers CSS pour ces guides. On s'y met (sur dev). C'est bon pour toi ?

lmoreau-gaudry commented 1 year ago

On vient de faire le jR6z

semiaddict commented 1 year ago

Avec Anne-Florence, on a regardé ça. Parmi ces exemples, seulement quatre guides (jR6z, OYgL, pYX1 et yPmW) sont publics. Le plus simple est de faire les modifications à la main des fichiers CSS pour ces guides. On s'y met (sur dev). C'est bon pour toi ?

Oui, c'est OK pour moi. Je pense aussi pouvoir mettre en place un script pour mettre ces guides à jour automatiquement (en convertissant les valeurs de rem en pixels) si c'est plus simple pour vous.

lmoreau-gaudry commented 1 year ago

Merci, c'est pas un gros travail, on va faire ça à la main.

semiaddict commented 1 year ago

C'est en place sur dev.

@lmoreau-gaudry, jR6z semble encore contenir des valeurs en rem.

lmoreau-gaudry commented 1 year ago

C'est une coquille de ma part, nous avons corrigé le pYX1. Les trois autres seront corrigé dans la semaine.

lmoreau-gaudry commented 1 year ago

J'ai mis à jour les feuilles CSS et lorsque je publie, j'obtiens l'erreur suivante : error_css

lmoreau-gaudry commented 1 year ago

Les valeurs sont bien passé en pixel dans l'édition de la page du cms, mais il n'y a pas de mise à jour, ni dans l'éditeur, ni dans le player.

semiaddict commented 1 year ago

J'ai mis à jour les feuilles CSS et lorsque je publie, j'obtiens l'erreur suivante :

@lmoreau-gaudry, peux-tu me donner les étapes de reproduction ?

lmoreau-gaudry commented 1 year ago
  1. J'ai ouvert directement la page : https://dev-metascore.philharmoniedeparis.fr/fr/app/pYX1/edit
  2. J'ai mis à jour les valeurs du bloc de texte "CSS"
  3. J'ai cliqué sur "Enregistrer pour partager" Mon navigateur : Chrome 111 sur Windows 10
semiaddict commented 1 year ago
  1. J'ai ouvert directement la page : https://dev-metascore.philharmoniedeparis.fr/fr/app/pYX1/edit
  2. J'ai mis à jour les valeurs du bloc de texte "CSS"
  3. J'ai cliqué sur "Enregistrer pour partager" Mon navigateur : Chrome 111 sur Windows 10

Merci @lmoreau-gaudry. C'est résolu.

lmoreau-gaudry commented 1 year ago

Je rencontre un souci avec la CSS de certains guides (comme jR6z), qui modifient la taille de police au niveau de l'élément <html>, car cela impacte toutes les mesures en rem dans la page. rem est une unité de mesure relative qui est basée sur la taille de police de la racine (l'élément <html>). Donc si je remplace html par .metaScore-app dans la CSS, toutes les mesures en rem ne correspondront plus à ce qui est prévu par l'auteur.e.

Savez-vous pourquoi il y a des paragraphes dans les applications qui utilisent rem pour la taille de la police ? D'après une recherche rapide dans la BDD, cela semble concernait 18 applications:

* [R65L](https://metascore.philharmoniedeparis.fr/app/R65L) "Contrastes  [copy Oussama]"

* [rkMw](https://metascore.philharmoniedeparis.fr/app/rkMw) "Contrastes Préprod"

* [o2Oj](https://metascore.philharmoniedeparis.fr/app/o2Oj) "La Course (EN), d’après Le Carnaval des animaux de Saint-Saëns [copy]"

* [n5xE](https://metascore.philharmoniedeparis.fr/app/n5xE) "LA COURSE / GUIDE DEFINITIF SIG [PRE PROD]"

* [NkOp](https://metascore.philharmoniedeparis.fr/app/NkOp) "LA COURSE / GUIDE DEFINITIF SIG [PRE PROD] [SANS LANGUE]"

* [L9ZA](https://metascore.philharmoniedeparis.fr/app/L9ZA) "La Course, d'après Le Carnaval des animaux de Saint-Saëns-OLD"

* [OYgL](https://metascore.philharmoniedeparis.fr/app/OYgL) "La Course, d’après Le Carnaval des animaux de Saint-Saëns"

* [jR6z](https://metascore.philharmoniedeparis.fr/app/jR6z) "Les nuances dans West Side Story"

* [pYX1](https://metascore.philharmoniedeparis.fr/app/pYX1) "Les textures dans le Sacre du printemps"

* [9rKx](https://metascore.philharmoniedeparis.fr/app/9rKx) "Les textures dans le Sacre du printemps [copie - test oussama]"

* [G6Qy](https://metascore.philharmoniedeparis.fr/app/G6Qy) "OLD-Les nuances dans les Danses symponiques de West Side Story"

* [KrQR](https://metascore.philharmoniedeparis.fr/app/KrQR) "Sacre PréProd"

* [PNjW](https://metascore.philharmoniedeparis.fr/app/PNjW) "Sacre, Sigrid-AF-V1"

* [0R8K](https://metascore.philharmoniedeparis.fr/app/0R8K) "Sacre, Sigrid-AF-V2"

* [kR2E](https://metascore.philharmoniedeparis.fr/app/kR2E) "Sacre, Sigrid-AF-V3-TEST BALLE PAYSAGE"

* [W6QJ](https://metascore.philharmoniedeparis.fr/app/W6QJ) "Sacre, Sigrid-AF-V4"

* [kR4E](https://metascore.philharmoniedeparis.fr/app/kR4E) "SAUVEGARDE-Les nuances dans West Side Story [copy]"

* [yPmW](https://metascore.philharmoniedeparis.fr/app/yPmW) "The Race, from The Carnival of the Animals by Camille Saint-Saëns"

C'est bon c'est corrigé.

semiaddict commented 1 year ago

Merci. J'ai basculé les correctifs de jR6z, OYgL sur prod, et j'ai aussi corrigé yPmW.