Rework search results details

arnaudvergnet commented 10 months ago

Creates new component using current state instead of loading whole document.

TODO:

[x] handle letters transcriptions
[x] let users open details even when there is no text search
[x] handle search highlights on arguments and letter transcriptions

Closes https://github.com/chartes/lettres-vue/issues/100

arnaudvergnet commented 10 months ago

@vicpsl normalement tout est bon.

vicpsl commented 10 months ago

Bonjour Arnaud, merci beaucoup. J'ai fait quelques corrections (affichage différencié en fonction du type de recherche notamment, suppression de la fonction highlight pour la transcription). J'ai poussé le code sur une nouvelle branche logilab-list-details https://github.com/chartes/lettres-vue/tree/logilab-list-details. Il conviendrait d'apporter les modifications suivantes également :

Général :
[x] corriger l'icône eye-btn dans DocumentListDetails : l'attribut :disabled="!witness.manifest_url" fonctionne sur la page /document/738 (deuxième témoin Orig. Bibl. nat. Fr., Nouv. Acq. Fr., 231, fol. 16.) :

mais pas sur DocumentListDetails : l'opacité n'est pas bonne etc et le cursor a dû être modifié manuellement dans le css pour .eye-btn

[x] réduction de la taille du guillement
[x] alignement du guillement à gauche sur la gauche du n° de lettre
[x] aligner le contenu à gauche (texte de transcription, analyse, etc) sur la gauche des dates (voir dev)
[x] ne pas aller à la ligne après [...] de chaque highlight (voir dev)
Responsive :
[x] s'assurer que l'icône guillement ne soit pas tronquée (voir dev)
[x] réduire la taille du texte (voir dev)

arnaudvergnet commented 10 months ago

@vicpsl j'ai cherry-pick tes commits, tu peux supprimer la branche logilab-list-details.

arnaudvergnet commented 10 months ago

Tous les points devraient être corrigés. Si c'est bon pour toi @vicpsl je peux nettoyer l'historique des commits.

vicpsl commented 10 months ago

Bonjour Arnaud,

merci beaucoup, les alignements à gauche entre guillemets/n° de lettre et texte/date ne sont pas tout à fait exacts, c'est beaucoup mieux et suffisant à ce stade.

Tu veux nettoyer l'historique avant que je fasse un merge ou je peux le faire ?

Merci

arnaudvergnet commented 10 months ago

nettoyage des commits terminé

arnaudvergnet commented 10 months ago

merci beaucoup, les alignements à gauche entre guillemets/n° de lettre et texte/date ne sont pas tout à fait exacts, c'est beaucoup mieux et suffisant à ce stade.

L'architecture du code rend cette tâche très complexe. Si on veut un alignement parfait il faudra passer plus de temps.

chartes / lettres-vue

Rework search results details #112