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

Ajout de quelques emojis sur le nouvel éditeur #5728

Merged
merged 4 commits into from Jun 6, 2021

Conversation

firm1
Copy link
Contributor

@firm1 firm1 commented Apr 19, 2020

Maintenant que c'est possible d'avoir des dropdowns dans la barre d'outil, on peut maintenant réintégrer les emojis qui n'étaient pas présent.

Ci-dessous une capture du résultat.

Capture d’écran de 2021-05-18 02-36-52

NB : les icônes de l'éditeur ne pouvant provenir que de fontawesome, je n'ai intégré uniquement les emojis dont la correspondance existait du coté de fontawesome.

PS : j'en ai profité pour rajouter le bloc attention (qui avait lui aussi disparu).

Contrôle qualité

  • Installez le site, et builder le front (make build-front sous linux)
  • Lancez rendez-vous sur l'éditeur de texte markdown et vérifiez que les boutons emojis fonctionnent comme attendu.

@Arnaud-D
Copy link
Contributor

❌ QA KO.

Les boutons se déroulent, mais le smiley est inséré presque jamais. Je clique et je clique, un peu sur tous les boutons, en alternant avec autre chose ou pas. Parfois le bouton marche, mais en général ça ne fait rien. Pareil pour les deux modes (normal et compatibilité). Pareil pour tous les boutons déroulants, y compris celui des blocs spéciaux.

Ceci dit, quand ça marche c'est le bon smiley et le bon bloc !

Je n'ai pas bien compris d'où vient le problème (un mélange de timing et de perte de focus ?), j'arrive pas à reproduire de manière systématique les cas de réussite. ☹️ En fait, c'est un problème déjà présent sur dev (je viens de revérifier), mais j'ai l'impression que c'est pire sur cette PR.

@firm1
Copy link
Contributor Author

firm1 commented Apr 19, 2020

Merci pour la QA. Je prendrai en compte les retour de cette PR un peu plus tard, puisqu'elle n'est pas si prioritaire.

@A-312
Copy link
Contributor

A-312 commented Apr 20, 2020

A l'occasion corrigé le background blanc (sur gris) de l'icone du dropdown serait bien x) J'avais déjà réfléchi à cette question pour avoir des icones customs pour les emoji, il faudrait qu'on remplace le contenu du dropdown avec notre propre contenu 🤔

@Arnaud-D Arnaud-D added this to À retoucher in Suivi des PR Apr 21, 2020
@A-312
Copy link
Contributor

A-312 commented Apr 23, 2020

J'y repense, on ne peut pas tricher en utilisant le menu des smileys de l'ancien éditeur vu qu'on a le code à disposition ?

@firm1
Copy link
Contributor Author

firm1 commented May 18, 2021

Je fais renaitre cette PR de ces cendres, avec l'utilisation des smileys zds (cf. capture).

A noter que j'en ai profité pour corriger le fond blanc derrière les sous menus.

@firm1 firm1 moved this from Modification demandée to En attente de QA in Suivi des PR May 18, 2021
@coveralls
Copy link

coveralls commented May 18, 2021

Coverage Status

Coverage remained the same at 86.652% when pulling 92164ec on firm1:add-buttons-to-editor into 7de5710 on zestedesavoir:dev.

Suivi des PR automation moved this from En attente de QA to Fusionnable après rebase Jun 6, 2021
Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

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

QA OK ✔️

J'arrive à mettre des smileys sans problème, ils sont corrects. Le fond du bouton des blocs est correct aussi.

J'ai toujours le problème d'insertion sur les blocs, mais ce n'est pas un bug introduit par la présente PR, donc indépendant de tout ça.

@Arnaud-D Arnaud-D enabled auto-merge (squash) June 6, 2021 19:02
@Arnaud-D Arnaud-D merged commit 427257a into zestedesavoir:dev Jun 6, 2021
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Jun 6, 2021
@artragis artragis added this to Done in Objectif v30.6 Jun 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Suivi des PR
  
Fusionnée
Development

Successfully merging this pull request may close these issues.

None yet

4 participants