maoschanz / emoji-selector-for-gnome

This extension provide a popup menu with some emojis ; clicking on an emoji copies it to the clipboard.
GNU General Public License v3.0
475 stars 76 forks source link

Search not working #185

Open q-wertz opened 1 year ago

q-wertz commented 1 year ago

System

Description of the bug

Is your bug already reported? [y/n]: n

To Reproduce

Steps to reproduce the bug:

  1. Install and activate the extension
  2. Click on the menu entry, start typing e.g. heart
  3. Nothing happens

Expected behavior

rapenne-s commented 1 year ago

I can reproduce it as well on OpenSuse MicroOS using https://github.com/maoschanz/emoji-selector-for-gnome/pull/186 on Gnome 44

VortexAcherontic commented 1 year ago

That’s strange as for me, also MicroOS w/ Gnome 44 the extension works just fine including the search: https://www.youtube.com/watch?v=AM0-bmpZTBc

Maybe some other extension breaking the behaviour for you?

Could you guys please provide some logs? I may try to incorporate a fix in my PR #186

You can run sudo journalctl -f in a terminal and then do the actions inside the extension to get some logs. It should output the errors there.

Edit: Are you running Wayland or X11?

rapenne-s commented 1 year ago

That's what I get when using the search, on wayland, and also on X

avril 12 15:31:25 t470 gnome-shell[1521]: JS ERROR: Exception in callback for signal: open-state-changed: TypeError: results.includes is not a function
                                          searchEmoji@/home/solene/.local/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiCategory.js:131:16
                                          _getResults@/home/solene/.local/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:116:36
                                          _onSearchTextChanged@/home/solene/.local/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:85:18
                                          _onSearchTextChanged@/home/solene/.local/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/extension.js:254:19
                                          clearCategories@/home/solene/.local/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/extension.js:249:8
                                          _onOpenStateChanged@/home/solene/.local/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/extension.js:151:8
                                          _callHandlers@resource:///org/gnome/gjs/modules/core/_signals.js:130:42
                                          _emit@resource:///org/gnome/gjs/modules/core/_signals.js:119:10
                                          open@resource:///org/gnome/shell/ui/popupMenu.js:927:14
                                          toggle@resource:///org/gnome/shell/ui/popupMenu.js:800:18
                                          vfunc_event@resource:///org/gnome/shell/ui/panelMenu.js:139:23
q-wertz commented 1 year ago

I am using Wayland.

Having the similar error as @rapenne-s

Apr 12 16:11:58 hoth gnome-shell[2175]: JS ERROR: TypeError: results.includes is not a function
                                        searchEmoji@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiCategory.js:131:16
                                        _getResults@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:116:36
                                        _onSearchTextChanged@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:85:18
Apr 12 16:11:58 hoth gnome-shell[2175]: JS ERROR: TypeError: results.includes is not a function
                                        searchEmoji@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiCategory.js:131:16
                                        _getResults@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:116:36
                                        _onSearchTextChanged@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:85:18
Apr 12 16:11:58 hoth gnome-shell[2175]: JS ERROR: TypeError: results.includes is not a function
                                        searchEmoji@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiCategory.js:131:16
                                        _getResults@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:116:36
                                        _onSearchTextChanged@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:85:18
Apr 12 16:11:58 hoth gnome-shell[2175]: JS ERROR: TypeError: results.includes is not a function
                                        searchEmoji@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiCategory.js:131:16
                                        _getResults@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:116:36
                                        _onSearchTextChanged@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:85:18
Apr 12 16:12:01 hoth gnome-shell[2175]: JS ERROR: Exception in callback for signal: open-state-changed: TypeError: results.includes is not a function
                                        searchEmoji@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiCategory.js:131:16
                                        _getResults@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:116:36
                                        _onSearchTextChanged@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/emojiSearchItem.js:85:18
                                        _onSearchTextChanged@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/extension.js:254:19
                                        clearCategories@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/extension.js:249:8
                                        _onOpenStateChanged@/usr/share/gnome-shell/extensions/emoji-selector@maestroschan.fr/extension.js:151:8
                                        _emit@resource:///org/gnome/gjs/modules/core/_signals.js:114:47
                                        close@resource:///org/gnome/shell/ui/popupMenu.js:944:14
                                        _onCapturedEvent@resource:///org/gnome/shell/ui/popupMenu.js:1390:18

Additionally I get errors from the org.gnome.shell.extensions.system-monitor extension (maybe related, just here for completeness)

Apr 12 16:11:52 hoth gnome-shell[2175]: JS ERROR: Error: GSettings key tooltip-delay-ms not found in schema org.gnome.shell.extensions.system-monitor
                                        _checkKey@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:733:23
                                        createCheckedMethod/<@resource:///org/gnome/gjs/modules/core/overrides/Gio.js:685:30
                                        on_leave@/usr/share/gnome-shell/extensions/system-monitor-next@paradoxxx.zero.gmail.com/extension.js:885:55
VortexAcherontic commented 1 year ago

Guys, I think I loosing my mind 😅 Tested this extension on two different machines, well kinda, both MicroOS and Gnome 44, but fail to get the same errors. I even triple checked I am running the same extension but I'm pretty sure I do.

Are there any other extensions you run? I mean it can't be that 3 people have the exact same issue while I struggle to reproduce it especially since @rapenne-s runs MicroOS as well.

rapenne-s commented 1 year ago

No other extension.

For installation, I cloned your git repo and run ./install.sh in it.

Could it be a missing font that would also contain more metadata or something like that?

VortexAcherontic commented 1 year ago

Yes I was finally able to reproduce the issue! 🥳

It seems the source of the actual extension available on extenstion.gnome.org differs a lot more from what is currently in the git-repo.

After installing the extension from the git-repo via the install.sh script I am now getting the same error. I used the latest release version and just patched the 44 into that one. It was my fault to expect the latest release to be the same as the latest git-repo commit.

VortexAcherontic commented 1 year ago

Deployed a fix which is now included in PR #186

q-wertz commented 1 year ago

Thanks a lot for your work :blush: