search

chartes / adele-vue

Site de l’application Adele (édition diplomatique collaborative)
2 stars 1 forks source link

[Analyse] Bugs d'affichage Mirador #14

Closed MrGecko closed 2 years ago

MrGecko commented 2 years ago

L'intégration de Mirador dans nos projets Vuejs pose quelques soucis, semblerait-il de css.

Bug 1 : Bien souvent au chargement d'une page, l'UI de navigation de la visionneuse n'est pas affiché correctement : image Dans ces cas là, il faut recharger la page pour le bug disparaisse.

Bug 2 : L'image n'apparaît pas d'elle-même et il faut cliquer et se déplacer pour que l'affichage soit correct. image Le manifest est bien chargé mais si on ne clique pas on ne voit rien

Objectifs :

fbessou commented 2 years ago

J'ai envoyé une PR avec mon interprétation du premier problème, je me suis permis d'écrire le bout de code correspondant, étant donné que le diff est minime.

J'attends de voir si ça résout le second avant d'investiguer.

MrGecko commented 2 years ago

@fbessou après une période de tests sur nos différents environnements, le problème apparaît (hélas) toujours à l'identique.

fbessou commented 2 years ago

Je viens de réussir à reproduire en local en compilant en mode production. Le problème arrive si je passe par un npm run build ou par un npm run serve -- --mode=production.

fbessou commented 2 years ago

L'implémentation de viewer.unmount utilise document.getElementById pour trouver l'élément conteneur sur lequel est monté le viewer. Mais en production, beforeDestroy est appelé après que vuejs ait détaché le composant du document. J'ai créé l'issue https://github.com/ProjectMirador/mirador/issues/3533.

fbessou commented 2 years ago

J'ai créé une PR sur Mirador qui corrige le problème. J'envoie une PR sur adele vue en attendant que celle de mirador soit intégrée: https://github.com/chartes/adele-vue/pull/22