Replace emoticons with emojione.com

[dbrugne]

Remove all server side emojis handling. Implement a brand new emojis selector and event rendering on Web client. Create migration task. On mobile ... do nothing.

http://emojione.com/developers

• [x] Mount library in project
• [x] Remove old code and implement new emojis picker in input rollup
• [x] Remove server-side code and control message length limit logic
• [x] Event rendering
• [x] Sprite on emojis rollup
• [x] Restore edit message logic
• [x] Handle on desktop notification
• [x] Mobile integration

Additional:

• [ ] Add entities filter in rollup (room/group, user) #918
• [ ] Add emojis keyboard navigation in rollup #1241
• [ ] Add minimal Ascii conversion (http://git.emojione.com/demos/2.1.0/ascii-smileys.html) => no, to many side effect and annoyed replacements

[hageged]

@dbrugne

• si je tape un mot, que je ne fais pas d'espace après ce mot, puis sélectionne un avatar, ce mot est remplacé par l'avatar. Il faudrait que l'avatar se mette à la suite et non à la place.
• Il y a un brave émoticone tout seul en haut de liste:
• taper sur Entrée devrait fermer aussi la pop-in Emoji. Parce que si on tape : puis du texte et qu'on appuie sur Entrée pour poster le message on se retrouve bloqué avec la pop-in ouverte
• ce serait bien de pouvoir naviguer avec les flèches et valider par Entrée
• ce serait bien que la pop-in reste ouverte quand on tape notre message si on l'a activée (et tant qu'on ne l'a pas désactivée par un clic, echap ou Entrée). Comme ça on pourrait taper plusieurs emoji à la suite (ce qu'on fait en général) c'est donc le comportement normal sur mobile.
• s'affiche pas encore sur le mobile (en revanche un code emoji tapé depuis le mobile s'affiche bien emoji sur le navigateur).

[dbrugne]

@hageged

• si je tape un mot, que je ne fais pas d'espace après ce mot, puis sélectionne un avatar, ce mot est remplacé par l'avatar. Il faudrait que l'avatar se mette à la suite et non à la place. => à corriger
• Il y a un brave émoticone tout seul en haut de liste: => c'est les catégories, fait CTRL+F5 tu dois pas avoir le fichier des icônes à jour
• taper sur Entrée devrait fermer aussi la pop-in Emoji. Parce que si on tape ":" puis du texte et qu'on appuie sur Entrée pour poster le message on se retrouve bloqué avec la pop-in ouverte => OK, il va probablement y avoir des effets de bord mais on va essayer. Si rollup emoji ouvert + 'enter' envoie le message (va fermer le popin)
• ce serait bien de pouvoir naviguer avec les flèches et valider par Entrée => oui en effet, c'est pourquoi c'est listé dans la tâche ci-dessus : "Add emojis keyboard navigation in rollup #1241"
• ce serait bien que la pop-in reste ouverte quand on tape notre message si on l'a activée (et tant qu'on ne l'a pas désactivée par un clic, echap ou Entrée). Comme ça on pourrait taper plusieurs emoji à la suite (ce qu'on fait en général) c'est donc le comportement normal sur mobile. => "quand on l'a activé" = pas clair : "quand on l'a ouvert en cliquant sur l'icône". Oui il faut maintenir l'état quand on ouvre le rollup emoji via l'icône jusqu'à ce que l'utilisateur le close ('esc' ou click sur fermer, poste le message ou re-click sur l'icône ou rollup de mentions/commande)
• s'affiche pas encore sur le mobile (en revanche un code emoji tapé depuis le mobile s'affiche bien emoji sur le navigateur). => sur iOS on a la saisie des emoji directement dans le clavier par défaut. Pas sur Android. Je n'ai rien prévu pour le moment sur Android. Il semble que Slack ai le même soucis. Nous n'ajouterons probablement pas la fonctionnalité sur Android avant longtemps.

[dbrugne]

Résumé des bugs/améliorations:

• [x] si je tape un mot, que je ne fais pas d'espace après ce mot, puis sélectionne un emoji, ce mot est remplacé par l'avatar
• [ ] si "enter" + rollup emoji ouvert doit poster le message finalement non, trop bizarre à l'usage, ça poste des emoji incomplets. Pour le moment on touche pas.
• [x] quand le rollup des emojis est ouvert depuis l'icône ne le fermer/remplacer que dans un des cas suivants:
  • [x] 'esc'
  • [x] 'enter' (post le message)
  • [x] re-click sur l'icône emoji
  • [x] click sur l'icône send
  • [x] click sur le lien close
  • [x] saisie déclenchant le rollup des mentions/commandes

[dbrugne]

@hageged C'est très compliqué à ajuster à tester comme composant car il y a un nombre de cas considérable tous basé sur des entrés utilisateurs (souris/clavier).

J'espère ne pas avoir introduit de régression ou autre problème fonctionnels.

On va se concentrer sur le bug fix sur ce ticket et plus d'amélioration si non bloquant.

[hageged]

@dbrugne

1)

quand le rollup des emojis est ouvert depuis l'icône ne le fermer/remplacer que dans un des cas suivants:

Ne fonctionne pas pour:

• entrée

je pense que c'est pas très grave et qu'on verra le rôle de la touche Entrée quand on fera #1241 EDIT: en fait c'est super relou dans le cas où on tape ":" suivi de quelque chose qui n'a rien à voir. Par exemple :) Là on se retrouve collé avec la pop-in qui est apparue sans qu'on le veuille et qui ne veut pas partir. A noter que si on tape un espace la pop-in se ferme. Ca concerne donc principalement le cas où on tape un smiley à l'ancienne en fin de post (donc pas de raison de taper espace après)

2) Bug un peu plus chiant en revanche mais on peut vivre avec pour une v1: lors de la sélection d'un emoji, le curseur ne se positionne pas après l'émoji, il reste là où il était (c'est à dire avant l'emoji). Du coup si on sélectionne un deuxième émoji il vient se mettre avant le premier au lieu d'être après.

[hageged]

Rien n'a changé. Donc reprendre mon commentaire précédent.