Preview : superposition de la toc avec l'article lorsqu'on ouvre le volet annotation

lakonis commented 1 year ago

Bonjour! petit bug d'affichage sur la preview:

Il faut sans doute rendre la règle css plus robuste :

  @media only screen and (max-width: 1000px)
nav {
  display: none;
}

ggrossetie commented 1 year ago

Je viens de tester sur Firefox et Chrome et ça semble déjà être le cas : responsive

lakonis commented 1 year ago

Merci pour l'anim :)

Je tente la mienne, depuis Firefox/Manjaro. Initialement je pensais que c'était parce que j'appliquais un zoom de 160% sur l'onglet Preview, mais ca donne ça à 100%:

Capture vidéo du 2023-06-28 09-45-49.webm

ggrossetie commented 1 year ago

On dirait que c'est le volet droit (annotations) qui se comporte différemment.

Est ce que tu as configuré une option d'affichage en particulier ? À partir d'une certaine largeur le volet ne se superpose plus au texte et prend trop de place pour que la table des matières s'affiche correctement.

ggrossetie commented 1 year ago

Je viens de trouver, une fois connecté le volet Hypothesis se comporte différemment 😬

lakonis commented 1 year ago

bien vu! Je n'y aurais pas pensé, et je suis souvent connecté par défaut sur hypothesis, donc difficile à comparer.

ggrossetie commented 1 year ago

Il y a plusieurs façons de corriger :

On masque quand la largeur est insuffisante
On passe la table des matières en overlay Dans ce cas je pense qu'il faut rajouter un bouton permettant de replier le TOC
On bascule le toc au dessus du texte
On force Hypothesis à être en overlay

En sachant que sur mobile il faudra masquer le TOC

lakonis commented 1 year ago

Je serais pour la seconde solution, toc en overlay si trop étroit, avec un bouton pour masquer/afficher.

Merci!

Le lun. 3 juil. 2023 à 17:34, Guillaume Grossetie @.***> a écrit :

Il y a plusieurs façons de corriger :

On masque quand la largeur est insuffisante

On passe la table des matières en overlay [image: image] https://user-images.githubusercontent.com/333276/250598777-0c5ec08d-ce07-4055-b5a8-b15aec69ae32.png Dans ce cas je pense qu'il faut rajouter un bouton permettant de replier le TOC

On bascule le toc au dessus du texte [image: image] https://user-images.githubusercontent.com/333276/250599212-cef771e8-ec7f-4e10-b47b-c7e22dfcff3c.png

On force Hypothesis à être en overlay

En sachant que sur mobile il faudra masquer le TOC

— Reply to this email directly, view it on GitHub https://github.com/EcrituresNumeriques/stylo/issues/907#issuecomment-1618687033, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACIKM3GFZ4JCPJDQHLNGARTXOLRBHANCNFSM6AAAAAAZVVI2SI . You are receiving this because you authored the thread.Message ID: @.***>

ggrossetie commented 1 year ago

Je viens de me rendre compte que c'est le template HTML qui génère cette page : https://framagit.org/stylo-editeur/templates-stylo

Je pense qu'il faut voir avec @RochDLY ou @antoinentl pour corriger.

Côté Stylo, on rajouter:

<script type="application/json" class="js-hypothesis-config">
{
  "openSidebar": true
}
</script>
<script src="https://hypothes.is/embed.js" async></script>

RochDLY commented 10 months ago

Voir le ticket #51 sur le framagit des templates.

EcrituresNumeriques / stylo

Preview : superposition de la toc avec l'article lorsqu'on ouvre le volet annotation #907