zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

[EasyMDE] problème de fullscreen #5692

Closed A-312 closed 4 years ago

A-312 commented 4 years ago

La class fullscreen n'est pas appliqué sur la toolbar quand on essaye de mettre l'éditeur en plein écran sur le forum.

artragis commented 4 years ago

investiguer du côté de la fonction EasyMDE.toggleFullScreen et regarder ce qu'elle fait passer en fullscreen.

Le cas de reproduction réel c'est :

c'est le message "La version actuelle du contenu provient d'une sauvegarde de votre navigateur. Vous pouvez revenir à la version originale (du serveur) avec CTRL+Z ou en cliquant ici." qui génère le pb

A-312 commented 4 years ago

Hypothèse : Le fullscreen doit être appliqué via un .prev()

A-312 commented 4 years ago

https://github.com/Ionaru/easy-markdown-editor/blob/228fe91a7230cd2430112ca877e3c6b367f039ee/src/js/easymde.js#L312

Il faudrait PR un changement : toolbarElements.parent

firm1 commented 4 years ago

On a une explication du pourquoi le bug n'est pas visible sur la prod, mais sur la beta/branche dev ?

A-312 commented 4 years ago

On a une explication du pourquoi le bug n'est pas visible sur la prod, mais sur la beta/branche dev ?

Tu n'as pas dû essayer avec le message qui dit "attention vous avez une sauvegarde", l'erreur est présente.

firm1 commented 4 years ago

Je reprends la question du coup: on a une explication du pourquoi on a pas le même comportement du fullscreen sur la prod et sur la beta ?

Sur la prod

Je vais là : https://zestedesavoir.com/forums/sujet/4035/exploration-craft/#p107578

J'ai ça :

Capture d’écran de 2020-04-15 15-29-47

Je clique sur l'icone : Capture d’écran de 2020-04-15 15-30-29

J'ai ça :

Capture d’écran de 2020-04-15 15-30-35

Sur la beta

Je vais là : https://beta.zestedesavoir.com/forums/sujet/4035/exploration-craft/#p107578

J'ai ça :

Capture d’écran de 2020-04-15 15-29-01

Je clique sur l'icone Capture d’écran de 2020-04-15 15-28-01

J'ai ça :

Capture d’écran de 2020-04-15 15-28-28

A-312 commented 4 years ago

Il y a le textarea parallèle sur la bêta

A-312 commented 4 years ago

ça aurait bien de dire que l'éditeur fonctionne correctement ici : https://github.com/Ionaru/easy-markdown-editor/pull/180 (= de faire un petit Q/A avec le script d'exemple)

firm1 commented 4 years ago

ça aurait bien de dire que l'éditeur fonctionne correctement ici

Je ne suis pas sur de savoir comment tester la PR de l'upstream, ça manque d'instruction de QA.

Sinon, pourquoi garder ce ticket ouvert du coup ? Le problème pointé est corrigé et donc on a plus de regression sur la branche de dev sur ce point.

A-312 commented 4 years ago

Tu builds et tu essayes le fullscreen : https://github.com/Ionaru/easy-markdown-editor/tree/master/example