oxen-io / session-desktop

Session Desktop - Onion routing based messenger
https://getsession.org
GNU General Public License v3.0
1.54k stars 195 forks source link

Bad smiley UX design #2634

Open Shallowmallow opened 1 year ago

Shallowmallow commented 1 year ago

Problem

When you type a ":)" smiley at the end of a message and you have your mouse at the same place, naturally you would type enter to send the message But then it selects the ":(" smiley. It happened to me and to the people I'm talking to plenty of times.

Possible Solution

I think the enter should select the smiley only if the mouse has actually moved to it. Or just not show a list of smileys.

KeeJef commented 1 year ago

Auto focusing the emoji panel allows the user to keep their hands on the keyboard and select an emoji. So personally i would be against requiring the panel to be focused via a mouse click. This is also how Discord and Telegram work

KeeJef commented 1 year ago

So i think the primary issue here is that if the cursor is in the wrong position then it will hover the incorrect selection, then if enter is pressed that selection will be made. Instead maybe hover doesn't change the selection, only changes the background color, only clicking changes selection (and navigating with arrow buttons)

knot126 commented 1 year ago

There is a similarly annoying issue when trying to type :3 at the end of a message, where it will select the 3️⃣ emoji instead of just sending :3. I think it would be better to require that you either press the up/down arrow at least once or end the emoji name with another colon (e.g. :3: for the emoji) in order for enter to confirm an emoji selection. Otherwise it should just treat this as regular text.