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

curseur invisible sur guide #pggy #672

Closed afborneuf closed 1 year ago

afborneuf commented 1 year ago

Il s'agit du "curseur 2b" de la page 6 de la partition. Je n'arrive pas à comprendre pourquoi on ne le voit pas, il a une couleur, une épaisseur, les TC sont cohérents entre eux et avec les TC de la page, il n'est pas caché... J'espère que je ne ne passe pas à côté d'un truc tellement énorme que je ne le vois pas !!!

semiaddict commented 1 year ago

@afborneuf, Cela semble être dû au fait que le curseur en question a des images-clés qui sont positionnées en dehors de ses limites. Je suppose que cela est survenu suite à une modification de sa largeur après l'attribution d'images-clés.

Nous devrions probablement discuter de la manière de traiter ce type de cas lors de notre prochaine réunion. En attendant, il suffit de supprimer les images clés dans le JSON des composants. Je peux le faire si besoin.

afborneuf commented 1 year ago

Ah bien vu @semiaddict Oussama ! Comme c'est une curseur qui est exactement dans la même position que celui qui est en dessous (nommé "Curseur 1a"), j'ai fort probablement dupliqué le 1a, changé ses TCI et TCO et du coup j'ai perdu la visibilité des images-clés. J'aurais du les effacer avant de modifier les TC !

semiaddict commented 1 year ago

Je propose donc de trouver un moyen d'indiquer la présence d'images-clé (peut-être en ajoutant un badge de comptage sur le bouton): image

Et de permettre de supprimer toutes les images-clé via le menu contextuel.

Qu'en pensez-vous ?

rbailly commented 1 year ago

Oui aux 2 propositions. Serait-il possible « d’empêcher » l’existence d’images clés à l’extérieur du TCI-TCout ?

Le mer. 26 avr. 2023 à 07:12, Oussama Mubarak @.***> a écrit :

Je propose donc de trouver un moyen d'indiquer la présence d'images-clé (peut-être en ajoutant un badge de comptage sur le bouton): [image: image] https://user-images.githubusercontent.com/522064/234475435-9229749f-f8e6-4d6f-b38b-07eacbc3853a.png

Et de permettre de supprimer toutes les images-clé via le menu contextuel.

Qu'en pensez-vous ?

— Reply to this email directly, view it on GitHub https://github.com/philharmoniedeparis/metascore-library/issues/672#issuecomment-1522794991, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACNY42HGQ5DSLGIT5ZCBYZTXDCVFLANCNFSM6AAAAAAW43D22A . You are receiving this because you are subscribed to this thread.Message ID: @.*** com>

-- R.

semiaddict commented 1 year ago

Serait-il possible « d’empêcher » l’existence d’images clés à l’extérieur du TCI-TCout ?

Ici le problème est plutôt la présence d'images-clé avec des valeurs de positions X supérieures à la largeur du curseur. Donc rien à voir avec les TCI et TCO.

Des images-clé avec des valeurs temporelles qui sortent des limites des TCI-TCO seraient encore visibles à l'édition des images-clé, donc moins problématique.

rbailly commented 1 year ago

Ok autant pour moi, je n’avais pas bien compris

Le mer. 26 avr. 2023 à 08:41, Oussama Mubarak @.***> a écrit :

Serait-il possible « d’empêcher » l’existence d’images clés à l’extérieur du TCI-TCout ?

Ici le problème est plutôt la présence d'images-clé avec des valeurs de positions X supérieures à la largeur du curseur.Donc rien à voir avec les TCI et TCO.

Des images-clé avec des valeurs temporelles qui sortent des limites des TCI-TCO serait encore visible à l'édition des images-clé, donc moins problématique.

— Reply to this email directly, view it on GitHub https://github.com/philharmoniedeparis/metascore-library/issues/672#issuecomment-1522868982, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACNY42AJR6HOSQN5GAC7L2LXDC7STANCNFSM6AAAAAAW43D22A . You are receiving this because you commented.Message ID: @.***>

-- R.

afborneuf commented 1 year ago

Merci pour ces solutions Oussama ! C'est une alternative envisageable effectivement. Ceci dit, si l'on prend en compte qu'on n'arrive sur cette situation que lorsqu'on modifie les TC d'un curseur déjà doté d'images-clés (et pour reprendre "à l'envers" la suggestion de Rodolphe), pourrait-on envisager un dispositif (alerte pop-up ou autre) qui s'activerait dans le cas où on a déjà des images clés et que l'on change le TCI ou TCO du curseur de façon telle qu'au moins une image clé se retrouve hors du champ temporel du curseur ?

semiaddict commented 1 year ago

@afborneuf, comme évoqué plus haut, il ne s'agit en fait pas d'un problème temporel, mais spatial.

semiaddict commented 1 year ago

Je ferme ce cas en faveur de #676 et #677.