search

CNRS-LACITO / eastlingplayer

This is the official repository of the Eastling player. It is part of the Eastling suite: Easy Annotation and Synchronization Tool for linguists.
GNU General Public License v3.0
2 stars 0 forks source link

Souci calage affichage sur phrase ou mot précis (via dièse) #55

Closed sguillaume closed 3 years ago

sguillaume commented 3 years ago

Les DOIs des phrases ne pointent pas sur les phrases concernées mais tous pointent sur le texte au niveau du titre (ça ouvre la page au niveau du titre au lieu d'ouvrir la page sur la phrase citée)

ex : https://doi.org/10.24397/pangloss-0001471#S10

alexis-michaud commented 3 years ago

Oui, c'est quelque chose qu'on ne pouvait pas tester avec la maquette.

Il faut que l'URL générée par le résolveur DOI contienne l'information de l'ancre sur laquelle 'caler' le texte.

Dans cet exemple, c'est le #S10 en fin d'URL : https://pangloss.cnrs.fr/corpus/show?lang=en&mode=pro&oai_primary=cocoon-0260c314-f906-3cf2-8d74-b62d0905b321&oai_secondary=cocoon-b3bbd8b7-2c3b-339c-a380-f0e941d1662f#S10

Vous voyez l'idée ?

Pour tous détails, au cas où vous auriez besoin d'éléments de contexte : infos très détaillées ici

alexis-michaud commented 3 years ago

Tant qu'on y est, on signale aussi le souci à @m8nli9ht : il y a une info à passer au lecteur, pour qu'il se cale sur l'unité n du premier niveau de granularité : S pour les textes (TEXT), W pour les listes de mots (WORDLIST).

m8nli9ht commented 3 years ago

Le soucis c'est que l'on a un autre ID que S10 (généré par le player) pour cette phrase et que l'on récupére depuis le fichier annotations : image

Ce n'est pas la première fois qu'on a une demande pour ignorer un ID, la question est donc : doit-on tenir compte des ID de phrases et de mots renseignés dans les fichiers annotations ? Si non = numérotations incrémentales dans le player, Si oui = dans quel(s) cas et doit-on les afficher quelque part ?

alexis-michaud commented 3 years ago

Réponse : Non, on ne tient pas compte des ID de phrases et de mots renseignés dans les fichiers annotations. Ils peuvent être erronés : numérotation non continue si une phrase a été insérée manuellement (ou un mot pour les listes de mots).

On re-numérote en parcourant le XML (parsing), en re-numérotant de 1 à n. C'est cette notation qui est affichée (S1, S2...) dans l'interface.

Autrement dit : numérotations incrémentales dans le player, c'est bien ça ! C'est à cette numérotation que fait référence le suffixe ajouté au DOI, et non aux identifiants contenus dans le fichier XML.

m8nli9ht commented 3 years ago

Traité : positionnement sur l'ancre passée en URL. A livrer.

m8nli9ht commented 3 years ago

Livré, à tester.

alexis-michaud commented 3 years ago

Testé avec cet exemple : https://doi.org/10.24397/pangloss-0000658#S9

En gros : ça marche ! 👍

Après :

image

Ca c'est quelque chose qu'on avait déjà vu auparavant : voir ici. Pour l'instant, l'affichage 'en haut' se place sur le haut de la page... qui justement est occupé par le bandeau. Il faut décaler la position verticale, de façon à ce que le haut de la phrase visée ne soit pas caché par le bandeau. C'est même mieux d'avoir une certaine marge en haut : qu'on voie la fin de la phrase précédente (ou unité W pour les listes de mots), bref 'centrer' (un tout petit peu) la phrase visée.

Résumé par Edouard (cf. lien déjà cité ci-dessus) :

il faudrait que ce soit le player qui décale le scroll de 350px vers le bas par rapport à ce qu'il fait actuellement. @matthew : tu peux le faire ? Ce sera plus propre que si je viens rajouter du js pour décaler les cartes vers le bas.

m8nli9ht commented 3 years ago

traité (sauf le "bug - encore - inconnu")

alexis-michaud commented 3 years ago

Le "bogue - encore -inconnu" ayant été traité ce jour (victoire !), reste seulement la question ci-dessus :

un réglage à faire : sur la page (lien) : le haut de la phrase est caché par le bandeau.

image

Ca c'est quelque chose qu'on avait déjà vu auparavant : voir ici. Pour l'instant, l'affichage 'en haut' se place sur le haut de la page... qui justement est occupé par le bandeau. Il faut décaler la position verticale, de façon à ce que le haut de la phrase visée ne soit pas caché par le bandeau. C'est même mieux d'avoir une certaine marge en haut : qu'on voie la fin de la phrase précédente (ou unité W pour les listes de mots), bref 'centrer' (un tout petit peu) la phrase visée.

Résumé par Edouard (cf. lien déjà cité ci-dessus) :

il faudrait que ce soit le player qui décale le scroll de 350px vers le bas par rapport à ce qu'il fait actuellement. @matthew : tu peux le faire ? Ce sera plus propre que si je viens rajouter du js pour décaler les cartes vers le bas.

m8nli9ht commented 3 years ago

ok je regarde ça aujourd'hui !

alexis-michaud commented 3 years ago

Pour refaire le point : il y a 2 choses.

D'une part, le détail d'alignement vertical. Quand on affiche le mot 36, dans cet exemple, il est en partie recouvert par le bandeau. Mieux vaudrait l'afficher franchement plus bas, en laissant de la marge en haut. Résultat dans Chrome (sous Windows) : image

A la limite, l'affichage de la phrase souhaitée (ici la 36) commencerait un bon centimètre au-dessous du haut de la page affichée que ça serait visuellement mieux.

D'autre part, la résolution du DOI semble poser un problème intermittent. Quand on saisit un DOI, l'affichage se fait parfois (sous Windows au moins) au niveau du titre (début du document), et non à l'endroit souhaité. Il faut alors recharger la page pour accéder à l'endroit souhaité. Ca rappelle le souci (heureusement disparu) de page blanche du lecteur (Pangloss #141) : tout n'était pas encore en place dans le lecteur quand les contenus étaient 'envoyés', et ça restait en blanc. Le souci dans l'affichage avec un DOI avec la bonne granularité constituerait-il une régression suite aux changements du code ? Le souci est seulement intermittent, peut-être attendre de voir si d'autres utilisateurs le rencontrent ?

m8nli9ht commented 3 years ago

j'ai ajusté, à tester