I can’t find any issues in the Console, while clicking the emoji d-icon. With limited knowledge about how it’s implemented, I can’t spot the bug. Hopefully someone are … (I’ve tried to look through the repo, but didn’t catch anything - with my limited knowledge about building plugins for Discourse (!))
Have anyone found out what the issue might be? Nothing have changed in the codebase of the plugin for quite a while, so I would guess something in the discourse codebase changed that makes this happen (haven’t seen any obvious changes which I would think have any effect on it though…).
There are multiple updates of Discourse that need addressing in this plugin.
I’ve made a PR that can serve as a temporary fix. I’ll probably be condemned to fiery pits of Tartarus for the way the fix works, but it could serve as a stop gap @gdpelican
@Jite You seem keen. Would you like to learn a bit more about Discourse development and the way this plugin works to help maintain it? I can help you learn the ins and outs of the Discourse codebase.
I would have loved to take you up on that offer, but as of right now, I have a bit too much stuff on my hands to do so. I’ve made a few smaller plugins already, but my ruby experience is so tiny that I haven’t had the ability to dig down into the internals too much yet, hehe.
I will stick around though, so if I find more time for it, I will gladly let you know!
I will take a peak at the fix too, as the instance I use the plugin on really seems to like it
So I tried using your temporary fix branch and if you’re not using a limited set it brings up the emoji picker and spins. It also doesn’t let you close the picker window. It actually also breaks the emoji picker in the default editing window as well (brings it up way over to the right side of the screen and spins). The limited set emoji picker stays static on the page as you scroll so it’s stuck wherever it comes up, but otherwise that works. I tried on both the latest Chrome and Firefox on OSX.
I glanced at your code and I don’t see anything obvious. I don’t know the discourse framework though. My gut would be it’s something with the changes relating to onShow, but I don’t do a ton of frontend development. I may take a deeper look if I have time and if I come across anything I’ll report it one way or another.
It’s been tested in full and limited mode, and also on mobile. You can test it out here yourself:
That site gets automatically rebuilt every 24 hours (which updates it to the latest Discourse). So if you’re on tests-passed (the default branch) it’s a good idea to test out any custom plugins you’re using there before you update.
I’ve also added a .discourse_compatibility file to the plugin:
This means that if you’re running the stable branch of Discourse you can keep updating as normal and this plugin will stay pinned at a commit that works with the latest stable release. See further here:
One thing I’ve noticed (other than it’s working, yay) is that when you click the retort button it will snap the page so the emoji picker is on the top (latest Chrome, OSX), then if you scroll your page it moves the emoji picker to some weird places. Just wanted to report.