FelipeFTN / Emoji-Copy

😄 Emoji copy is a versatile extension designed to simplify emoji selection and clipboard management.
https://extensions.gnome.org/extension/6242/emoji-copy/
GNU General Public License v3.0
106 stars 12 forks source link

Emoji search returns wrong or no emojis #13

Closed illtellyoulater closed 10 months ago

illtellyoulater commented 12 months ago

Bug Report

Current Behavior Searching returns wrong emojis, or no emoji, for example searching "happy" shows: 😒🙋

Expected behavior/code I think it should return these ones 😊😄😀😃😁, perhaps even this one 💩 (😉)

Environment

Possible Solution I think your extension uses Gnome emoji system under the hood (which can be run by default with this sequence CTRL-SHIFT-e-SPACE), because it seems that when I search "happy" there, the same 🙋 is shown (and the wrong one 😒 is not shown). This little bug apart, it would be cool if we could do something to make the Gnome emoji system itself respond to more words, as your extension would benefit from this too.

Additional context/Screenshots n/a

FelipeFTN commented 12 months ago

Hello, friend! Thanks for reporting this bug! :100: Sorry for taking too long to reply :sweat:

Looks like it's true! I just tested it and "happy" is returning: 😒🙋 - Nice catch! I tested other words like: "sad", "smile" and even "surprise", they seem to work just fine! I highly agree that we should build a better searching system. The current one is pretty simple and archaic! I will keep this issue on track! Thank you so much for reporting!

Feel free to contribute or share, if you get any solution! :heart:

illtellyoulater commented 11 months ago

Will be more than glad to collaborate if I can! Your good vibes and kind words are highly appreciated! Keep up the great work! 🚀🦀

FelipeFTN commented 11 months ago

Thanks buddy! :tada: Looks like the file we need to change is this one: emoji-copy@felipeftn/data/emojisKeywords.js. Take a look at the word: "happy". You may see two emojis using this word: person raising hand (Line 270) and unamused face (Line 54). Looks like the best way to make a better emoji search filter, is by adding and changing the words in this file. ~maybe GPT can help too~ 😆 Do you want to work on this improvement? It seems to be a simple task, updating the keywords for each emoji on the emojisKeywords.js file. I will be working on the Gnome 45 migration before this. Feel free to open an PR to contribute!

q-wertz commented 11 months ago

Maybe one can also contact and reuse others work, e.g. https://github.com/github/gemoji (see field tags in db/emoji.json) (MIT License)

FelipeFTN commented 10 months ago

This problem was fixed both on #23 #18 Thanks to @helena-dev and @NatVIII :blue_heart: