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

sauvegarde de la couleur de remplissage pour les formes SVG de la bibliothèque #653

Closed afborneuf closed 1 year ago

afborneuf commented 1 year ago

Lorsque l'on assigne une couleur de remplissage à une forme SVG de la bibliothèque, cette couleur n'est pas toujours conservée à la réouverture du guide. Sur le guide #j2R4 par exemple, la couleur de remplissage des ronds bleus visibles à l'activation du lien "ré-annoncées avec l'arrivée du thème E" a été conservée, par contre celle des ronds qui devraient apparaître en orange en activant le lien "notes répétées" n'est pas restituée à la sauvegarde (bien qu'apparaissant toujours dans les paramètres). Il s'agit de l'élément "suivi-des-quintes" qui fait l'objet d'une animation. probablement en lien avec #645

semiaddict commented 1 year ago

Il me semble que ceci a été résolu en même temps que #645.

afborneuf commented 1 year ago

J'ai en effet vu ce bug réparé vendredi, mais aujourd'hui je constate que les couleurs de remplissage sont redevenues invisibles. Pour reprendre l'exemple ci-dessus, les ronds oranges qui font l'objet d'une animation n'apparaissent pas.

semiaddict commented 1 year ago

@afborneuf, peux-tu m'indiquer le(s) nom(s) ou identifiant(s) du/des composant(s) en question ? Personnellement, je vois les ronds oranges animés :

firefox_Lopg6I6hEF

afborneuf commented 1 year ago

Bon... alors ce doit encore être le fait de mon vieux windows... j'ai effectivement vu l'animation avec les ronds orange sur mon mac perso, mais aujourd'hui, sur mon PC de travail, je ne les vois pas.

afborneuf commented 1 year ago

Désolée Oussama ! ... Je ne vais plus oser déclarer de bugs.

semiaddict commented 1 year ago

Désolée Oussama ! ... Je ne vais plus oser déclarer de bugs.

Non, non, ne t'inquiète pas, cela ne me dérange pas du tout :)

Si cela ne se produit que sous Windows 7, j'aimerais savoir pourquoi. Penses-tu que nous pourrions organiser une session Teams pour essayer de comprendre cela ? Je suis disponible la majeure partie de cette semaine et la suivante.

afborneuf commented 1 year ago

Oui bien sûr Oussama ! Par exemple :

semiaddict commented 1 year ago

Est-ce que demain à 9h30 ça te convient ? N'oublie pas de m'envoyer un lien Teams (à moins que tu ne veuilles le faire sur Skype).

afborneuf commented 1 year ago

OK pour demain matin 9h30 (j'espère avoir trouvé d'ici là comment t'envoyer un lien :-)

semiaddict commented 1 year ago

Cela semble être un problème spécifique à Firefox sous Windows 7 similaire à #520.

Les valeurs CSS sont bien prises en compte par le navigateur (voir capture ci-dessous), mais elles ne sont pas reflétées dans le rendu graphique.

image

semiaddict commented 1 year ago

En regardant de plus près sur Browserstack, j'ai l'impression que ce ne soit pas la couleur qui n'est pas prise en compte, mais plutôt les dimensions de l'élément SVG.

image

Il est donc possible que ce soit un problème en provenance du script interne à l'SVG.

semiaddict commented 1 year ago

Je pense avoir trouvé la source du problème, et l'avoir corrigé sur dev. Il s'agissait en effet d'un problème dans l'interprétation du script intégré dans le SVG sous Firefox dans Window 7. J'ai donc mis à jour les formes SVG.

J'ai également corrigé un problème de positionnement de la balise <object> des SVG, qui faisait que les ronds étaient coupés en bas.

@afborneuf, peux-tu me confirmer que le rond orange apparaît bien sous Windows 7 dans https://dev-metascore.philharmoniedeparis.fr/fr/editor/ERJK (vers 00:00:32:90) ?

afborneuf commented 1 year ago

Tu es un as Oussama ! ça fonctionne ! Et merci pour les ronds coupés... jusqu'à maintenant, je croyais qu'ils étaient couverts par un autre élément !

semiaddict commented 1 year ago

Super. C'est en prod.

semiaddict commented 1 year ago

Note: il peut être nécessaire de vider le cache du navigateur pour voir le changement.