Ajouter une mention/emoji/tag ajoute un espace à la fin mais ne déplace pas le curseur

Lorsque vous ajoutez une mention ou un emoji en tapant @ / :, suivi de ce qui est nécessaire, puis en appuyant sur Entrée, le reste du nom d’utilisateur ou du nom de l’emoji est complété, suivi d’un espace. Je suppose que cet espace est là pour vous permettre de continuer à taper, mais il ne déplace pas le curseur pour tenir compte de l’espace nouvellement ajouté. Ainsi, tel que cela fonctionne actuellement, vous devez tout de même appuyer sur la barre d’espace, ce qui laisse un espace superflu à la fin.

Étant donné que vous voudrez peut-être aussi terminer une phrase (ou avoir fini d’écrire le message) après la mention ou l’emoji, je serais plutôt enclin à supprimer cet espace superflu et à laisser les utilisateurs appuyer sur la barre d’espace s’ils en ont besoin. La deuxième option consisterait à déplacer le curseur d’un caractère vers l’avant, pour qu’il se trouve après l’espace (était-ce déjà le comportement prévu ?), mais je ne vois pas vraiment de valeur ajoutée à cela. Je préfère appuyer sur la barre d’espace quand j’en ai besoin plutôt que de supprimer un espace ajouté automatiquement quand je n’en ai pas besoin.

(Je sais que cela frôle le bug, donc n’hésitez pas à déplacer ce message si cela relève davantage de ux ou de #feature. D’ailleurs, en écrivant la phrase précédente, je me suis rendu compte que cela se produit aussi avec les tags / #.)

4 « J'aime »

C’est assurément original, c’est légèrement agaçant. GitHub déplace le curseur, par exemple, ce qui a plus de sens.

Je soutiens le déplacement du curseur.

3 « J'aime »

Quelque chose a-t-il changé là-dessus au cours des deux dernières semaines environ ? :thinking:

J’ajoute beaucoup de valeurs :t2 à la fin de tous mes émojis et je les tape toujours au clavier.

Par exemple, pour obtenir un pouce vers le haut de couleur blanche, je tape :

:+

Ensuite, j’appuie sur tab pour compléter le pouce vers le haut jaune, puis je continue à taper :t, ce qui me présente une courte liste de couleurs :

À ce stade, j’appuie simplement à nouveau sur tab pour compléter automatiquement la variante :t2 que je souhaite.

Cependant, suite à ce qui semble être une mise à jour récente, chaque fois que j’appuie sur tab pour compléter le premier :+1:, un espace est maintenant ajouté à la fin, ce qui signifie que je dois maintenant appuyer sur la touche retour arrière, puis continuer avec mes variantes :t ?

1 « J'aime »

Presque certainement une nouvelle régression, peut-être que @sam pourra regarder quand il sera de retour.

2 « J'aime »

Je comprends tout à fait les inconvénients ici, mais auparavant, nous insérions un espace et reculions le curseur, ce qui créait de la confusion et engendrait de nombreux problèmes subtils.

Nous apprécions certainement d’insérer un espace, mais je suis réticent à ce que :heart: insère un espace tandis que :+1: ne le fasse pas.

Je pense que la meilleure solution serait d’introduire un « paramètre d’arrière-plan » dans les paramètres locaux du navigateur. Il suffirait que le client « se souvienne » du ton de peau sélectionné la dernière fois et l’ajoute par défaut.

La correction est alors triviale : vous ne modifiez cela qu’une seule fois ici :

Ainsi, les futurs emojis que vous insérerez incluront automatiquement le ton de peau, par exemple :+1:t3:. J’apprécie beaucoup cette approche car elle respecte davantage les utilisateurs. Il est très probable que vous souhaitiez conserver le ton de peau que vous venez de choisir.

@codinghorror, qu’en penses-tu ?

3 « J'aime »

Ça me semble bon ! :+1:t6:

3 « J'aime »

Ce PR devrait le faire :

L’application cliente se souvient déjà de la teinte de peau sélectionnée par l’utilisateur (et le sélecteur d’emojis respecte déjà la teinte de peau choisie), mais actuellement notre autocomplétion d’emojis ne respecte pas la teinte de peau sélectionnée et propose toujours la teinte de peau jaune. Ce PR fera en sorte que notre autocomplétion d’emojis utilise par défaut la teinte de peau sélectionnée par l’utilisateur.

Avant / Après :

4 « J'aime »

Incroyable !!

Un grand merci à tous, ce sont les petites choses qui font de cette plateforme un lieu aussi formidable :bowing_man:t2:

4 « J'aime »

@seanblue Es-tu satisfait de la nouvelle fonctionnalité ? Je vais la fermer automatiquement demain.

2 « J'aime »

Concernant ma plainte/suggestion initiale, oui, je suis satisfait du changement. Le curseur est maintenant décalé après l’espace inséré, donc je pense que c’est bon. Merci d’avoir pris en charge cela.

4 « J'aime »