Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Améliore la barre d'outils du nouvel éditeur #6138

Merged
merged 1 commit into from
Jun 17, 2021

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Jun 13, 2021

La barre d'outils est visible en plein écran et passe au dessus de la bannière d'information lorsqu'elle est présente. Lorsqu'elle n'est pas en plein écran et lorsqu'on n'est pas sur un téléphone, elle reste toujours à l'écran (sticky) même lorsqu'on écrit un texte très long.

Fixes #6137

QA :

  • source zdsenv/bin/activate && make update && make zmd-start && make run-back
  • Vérifier que la barre d'outils est visible en plein écran
  • Vérifier que la barre d'outils reste visible à l'écran même lorsqu'on écrit un texte très long

@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.689% when pulling 699091a on Situphen:editor into 836d6b8 on zestedesavoir:dev.

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rapport de QA

OK ✔️ la barre d'outils de l'éditeur est bien visible en plein écran.

Mais pourquoi est-ce que lorsque l'éditeur est en plein écran, on n'affiche la barre d'information seulement sur mobile ?

@Situphen
Copy link
Member Author

Merci pour la QA !

Mais pourquoi est-ce que lorsque l'éditeur est en plein écran, on n'affiche la barre d'information seulement sur mobile ?

Normalement, que ce soit mobile ou pas, lorsque l'éditeur est en plein écran le bandeau d'information n'est pas visible. Ou alors tu parles d'autre chose ?

@philippemilink
Copy link
Member

Mais pourquoi est-ce que lorsque l'éditeur est en plein écran, on n'affiche la barre d'information seulement sur mobile ?

Normalement, que ce soit mobile ou pas, lorsque l'éditeur est en plein écran le bandeau d'information n'est pas visible. Ou alors tu parles d'autre chose ?

Voilà ce que j'ai, avec deux tailles d'écrans différentes:
Capture d’écran_2021-06-15_19-53-50

@Situphen
Copy link
Member Author

En fait sur mobile le plein écran de l'éditeur est totalement buggé. Si tu es tout en haut de la page et que tu l'actives, tu as effectivement ce que l'on voit sur ta capture d'écran, mais dès que tu es un peu plus bas dans la page (la barre de défilement n'est pas tout en haut) alors on n'a plus accès à la barre d'outils. J'ai investigué et cela est dû à la propriété transforme: translate3d(0,0,0); de .page-container sur mobile, qui semble être utile pour faire fonctionner le menu mobile.

Je ne sais pas s'il y a moyen d'arranger les choses pour que ça fonctionne correctement sur mobile. Aussi, ça demanderait pas mal de recherche pour pas grand chose à mes yeux (car qui met en plein écran sur mobile ?).

@philippemilink
Copy link
Member

Donc je dirais qu'on peut merger.

@Situphen Situphen merged commit 0c9d904 into zestedesavoir:dev Jun 17, 2021
@Situphen Situphen deleted the editor branch June 17, 2021 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Bug d'affichage nouvel éditeur en plein écran
3 participants