-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add copy on Ctrl+C and context menus for each emoji #100
Comments
First of all, thanks for the gratitude. If you want to find more information on how to support us, have a look at https://github.com/rugk/awesome-emoji-picker/blob/master/CONTRIBUTING.md#support-us. 😃 That said, next time, please open a new issue for each idea/issue/bug you have, so they can be tracked separately.
This is already tracked in #41.
I'd also say this belongs to #41. However, I don't think this is very useful, because while Ctrl+C is well known for copying, it only applies when text is selected. You don't select an emoji with this add-on you pick it. 😜 No, srsly, you see no blue selection bar and thus I guess people won't get this would work. So UX-wise, I don't think this is useful.
Again the same topic, although your idea is interesting. That said, I'll still think it's quite intuitive to differentiate between the keys here, because how should a user know, which key does what (and what exactly one does not?). Also what should clicking then do? (So I don't agree on the UX aspects here.)
You can change this behaviour in the options.
This is more or less a workaround that has been introduced, because some sites don't properly work when trying to insert the emoji directly. See this label for known site issues.
Thanks for the use case. Indeed, I acknowledge this is a problem. At least you can partially (or totally) disable that behaviour in the settings (e.g. because of that reason). Thanks for the feedback anyway. Do you think #41 addresses all your concerns, so I can close this one in favor of the other one? |
But I don't want to disable copying. It's just a separate use case altogether. Have you used the WhatsApp desktop app? Or the Telegram desktop app? In either of these I can hit the Can you imagine if this erased my clipboard!??? 😮 In fact do you know how I just entered that emoji 😮 above, here on GitHub? I again hit So we see a pattern:
So there shouldn't be a need to do a "workaround" by going into the settings. Just don't insert anything into the clipboard when I hit But that doesn't mean I still don't need to insert the emoji into the clipboard sometimes. Your add-on is so good that I may want to switch to Firefox just to use the emoji picker, and then switch back to insert it into some other app. But that shouldn't be done by the standard key. There needs to be some other keypress for copying the emoji. What better than
That is an odd thing to say. If you don't select an emoji, how can you insert it? Of course you select it. You just use the focus indicator rather than the selection indicator.
That just means you need to improve the UI of your selection. Because you do have a selection, otherwise we couldn't know which emoji we were inserting! So you can either prevent us from using |
Reg shortkey insertion: Yes, I know this, actually you can use it on GitHub too, as you've noticed. That said, I totally see that these are different use cases that I just want to summary right now (from what I understood):
Going back to the Alone visually it looks more similar. However, in an emoji picker you can correct your result, search again and see all results etc. It's a slightly different UX experience. That said, there is one thing I agree on: They both usually don't copy an emoji to the clipboard.
Now, there are websites for the last thing too, but an add-on is obviously more convenient.
This is correct and already done as part of #41. But even if we'd give them a big blue background when we cycle through them, that does not change the fact that users don't expect that you can use a Ctrl+C short hot key to copy the emoji. All I want to say is: Users won't discover this feature on their own. That does not mean it could not be a good addition as a "power feature", because adding it breaks no user workflow, you cannot copy something else there at that point e.g. In the next comment I'll describe some ideas on how that may look. Off-topic, but also note that if you just want to copy an emoji to your clipboard you can also use the address bar. |
As such, I'd be fine with this idea if:
As for the new feature:
Whether both are technically possible in a good way I'm not yet totally sure, especially given we may need the new ¹ only for the "workaround use case" this may be impractical, so maybe an additional setting could help here, but actually I'd say that is not worth it. And… I'm of course happy to accept PR's for this feature. |
If #93 is merged and you type just |
When the current behavior is unexpected and destructive, sometimes it needs to be changed even if current users go through a little transition in order to prevent data loss and to gain even more users. Here are the considerations: This behavior is not consistent with what I would expect after using emoji selectors the most popular applications including Slack, WhatsApp, Telegram, etc. (You can try to make distinctions between emoji "selectors" and "pickers, but I don't buy it. What's important is not how you view the application, but how users are using it. Why not take a survey of they users?) The behavior is destructive: information is lost. Sure maybe your initial users expect that their clipboards will be erased. But do they like that? And what about new users coming in? Will they even bother finding the GitHub page and opening a ticket? Or will they just switch to a different emoji selector, oops, I mean "picker".
|
If users install the add-on it's clearly stated right in the first text they read (AMO summary e.g.) that this modifies the clipboard.
It is. That said: As stated before, I changed the scope of this issue and it is clear that I'd think adding Ctrl+C support is a nice feature to have (nice to have), so I do appreciate PRs. If you really want it, you could take a look at the contributing guide and add it. |
Blocked by missive/emoji-mart#444 (unless someone wants to create a custom implementation, as said, PRs are fine) |
Hi, I hope you're doing well. I haven't had a chance to use your great extension again until recently, on a laptop upgraded to Ubuntu 18.04. I don't know what the final status is with the ability to select with arrows and/or
I don't remember all the intricacies of our discussion regarding selection, so apologies if we already covered this. I just wanted to document the problem in case you thought you had fixed it since then. |
I filed this three years ago. I'm stopping by to check the status:
|
As said, still all blocked by missive/emoji-mart#444 and also kinda by #116, which in itself also has higher priority, but was kinda stuck on implementation (see PR). |
Edit (by @rugk): The scope of this issue has been redefined in #100 (comment)
This is a really good emoji picker! Congratulations and thank you!!
But it needs a few usability tweaks to make it really awesome.
Let's say that I hit
Ctrl+Shift+.
to bring up the picker and then type "joy". I am presented a set of options. It is at this point where the picker UI needs a little bit of usability improvement.Tab
. And after one of the options is selected, I should be able to use the arrow keys to navigation the options.Ctrl+C
: Once I the correct emoji is selected, I should be able to useCtrl+C
to copy the selected emoji to the clipboard. And why not?Ctrl+C
is the standard key combination for copying. Everywhere. For everything.Enter
: Once the correct emoji is selected, I should be able to hitEnter
to insert the selected emoji at the current point. But this should not copy it to the clipboard. The current behavior both copies and inserts the emoji. But I would seldom need to do both. If I insert it, why do I need it in my clipboard? And if I asked to copy it to my clipboard, then that probably means I didn't want to insert it (or I would have inserted it). The problem here is that I may be working on a large development project or typing a long description, and I have something important in my clipboard. I may just want to insert an emoji. But why should I lose everything in my clipboard just to do that?So thanks again for the great start on this emoji picker. I hope you can make it truly awesome; it's so close to being awesome already!
The text was updated successfully, but these errors were encountered: