Problème avec le filtre cutnotes

brrd commented 4 years ago

Le filtre cutnotes utilisé pour générer les notes marginales ne fonctionne pas correctement avec les documents importés en TEI.

Le comportement qui pose problème :

Les notes ne sont pas coupées à 150 caractères, malgré le [#NOTESBASPAGE|cutnotes(150)]
Le markup des notes n'est pas correct : les p portent la classe "texte" et non "notesbaspage" (voir ci-après)

Markup attendu :

<div id="sidenotes" class="article__sidenotes col-3 d-hidden d-md-visible">
  <p class="notesbaspage">...</p>
  ...
</div>

Markup obtenu :

<div id="sidenotes" class="article__sidenotes col-3 d-hidden d-md-visible">
  <p class="texte">...</p>
  ...
</div>

Exemple (en TEI) de note mal gérée :

<p>Mais l’inscription des ressources patrimoniales au rang des priorités du projet d’établissement 2012- 2015 de l’ABES<note place="foot" n="1"><p> Axe 5 : «Signaler les ressources des établissements ». Pour une approche détaillée des problématisations définies à l’occasion du projet d’établissement de l’ABES, nous renverrons notamment à l’article de Yann Sordet, « Le signalement du patrimoine de l’enseignement supérieur : une réflexion renouvelée », <hi rend="italic">Arabesques, </hi>n° 65, 2012.</p></note> doit aussi nous entraîner à faire abstraction d’un paradigme applicatif évidemment prégnant à l’Agence.</p>

Exemples en ligne :

Un exemple du problème : https://dev.edinum.org/novabis/338
Un exemple qui fonctionne : https://dev.edinum.org/nova/74

Code concerné :

Le filtre cuttext : https://github.com/edinum/lodel-textfunc/blob/92295a26b5942b37813cb9ce194e797934c75f61/notes.php#L16
L'endroit où il est appelé dans la nova : https://github.com/brrd/nova/blob/d4a13e48bbbf94b72fd088c1be444325555e5d59/macros_article.html#L499

Edit : TODO

[x] @ShazaamYo : corriger les sélecteurs des notes et sidenotes
[x] @brrd : corriger le JS des sidesnotes
[x] @arnaudcordier : modifier le filtre pour accepter toutes les possibilités

arnaudcordier commented 4 years ago

Le problème est que les

dans les notes n'ont pas de class 'notesbaspage' mais une 'texte'.

J'ai pris le truc en compte pour que ça cherche les p[class=texte] si il n'y a pas de notesbaspage.

Il faut revoir le CSS en conséquence, ou je peux forcer la class dans le filtre au besoin
N'est ce pas un problème de stylage du document ou de ME qui ne met pas bien les class comme il faut ?

brrd commented 4 years ago

je peux forcer la class dans le filtre au besoin

J'ai l'impression que c'est finalement ce que tu as fait sur https://dev.edinum.org/novabis/338 car je vois que le style dans les notes marginales est devenu notesbaspage.

J'ai chargé le même XML dans une lodelia et le style reste texte dans les notes de bas de page, mais cela ressort bien dans la maquette : http://checklist.labocleo.org/test/index.php?id=867

Du coup je suppose que le mieux est de faire pareil qu'OpenEdition et d'agir sur les CSS plutôt que considérer que c'est un problème de stylage ou de ME.

Du coup je vous propose de :

@ShazaamYo : remplacer les sélecteurs .notesbaspage par .article__footnotes-contents p et .article__sidenotes p pour que l'affichage accepte toutes les class
@arnaudcordier : ne pas toucher au filtre

brrd commented 4 years ago

...et ça veut aussi dire qu'il faut moi aussi que je corrige le js !

arnaudcordier commented 4 years ago

Si il faut quand même toucher au filtre, sinon ça ne cut pas le texte. Mais je ne touche pas à la class du

Le filtre fait donc: Si il n'y a pas de class=notesbaspage, alors chercher des class=texte

brrd commented 4 years ago

Le filtre fait donc: Si il n'y a pas de class=notesbaspage, alors chercher des class=texte

AMA il faut plutôt chercher tous les p qui sont dans le container de notes indépendamment de leur class. Apparemment la lodelia supporte maintenant plein de class dans les notes : citations, légende, etc.

arnaudcordier commented 4 years ago

Voilà, ça prend les éléments à la racine comme des notes. C'est donc complètement générique.

Il faut changer le CSS pour que ça ne tape pas sur p.notesbaspage mais sur ce qui est dans la colonne notes

=> https://dev.edinum.org/novabis/338

ShazaamYo commented 4 years ago

ok j'ai remplacé les sélecteurs .notesbaspage par .articlefootnotes-contents p et .articlesidenotes p
sauf .notesbaspage__more-link, car je ne sais plus a quoi ca correspond

brrd commented 4 years ago

@ShazaamYo .notesbaspage__more-link doit être traité de la même façon que le reste. C'est un lien "voir les autres notes" qui s'affiche quand il n'y a plus de place dans la marge pour afficher d'autres notes.

ShazaamYo commented 4 years ago

et ca deviens quoi du coup ? .article__footnotes-contents p.more-link ? y a un exemple qq part ?

brrd commented 4 years ago

et ca deviens quoi du coup ?

<p class="notesbaspage notesbaspage--more">
  <a class="notesbaspage__more-link" href="#ftn10">@notes_suivantes</a>
</p>

y a un exemple qq part ?

https://dev.edinum.org/nova/306

ShazaamYo commented 4 years ago

ok merci

arnaudcordier commented 4 years ago

Donc on ferme le ticket non ?

brrd commented 4 years ago

Nope, je dois encore vérifier que le JS qui positionne les notes fonctionne indépendamment de la classe des p. Je fais ça mardi prochain quand j'ai du temps.

brrd commented 4 years ago

Normalement c'est ok.

chapitreneuf / nova

Problème avec le filtre cutnotes #39