[nodebb-plugin-emoji-extended] Emoji Extended

frissdiegurke

seems like encoded ULIs get filtered to 404 o_O I guess this needs to be fixed within core...

julian

@frissdiegurke Take a look at this autofill.js:

nodebb-plugin-mentions/static/autofill.js at df358c2924f7dc1417c29acc4c1df5dfe8749b3d · julianlam/nodebb-plugin-mentions

NodeBB Plugin that allows users to mention other users by prepending an '@' sign to their username - nodebb-plugin-mentions/static/autofill.js at df358c2924f7dc1417c29acc4c1df5dfe8749b3d · julianlam/nodebb-plugin-mentions

GitHub (github.com)

I'm listening for the client-side event action:composer.loaded, which is better than adding a new textcomplete onfocus

julian

I'm thinking of bundling emoji-extended with all new NodeBB installs... :shipit:

Two issues however:

Emoji autocomplete does not seem to work when editing a post (my post before this one might help with that)
Typing : triggers autocomplete even at the end of words (perhaps it should check for a word boundary)
- It also means autocomplete triggers when I am typing the end of another emoji: :shipit: <-- autocomplete triggers

frissdiegurke

@julian First issue should be fixed already since I've used your solution like 10min after your post .
Second issue already noticed, will be fixed soon...

(if there would be no 404 Error on this) to bundling

New issue figured out writing this post: some mappings (just some because of other reasons) need \s|$ after key, so I can't write sth. like :P. ^^

Ted

Hey guys,

I'm testing this plugin. Sounds great but I can't make the suggestion scroll list appears like on this forum.

I'm having the default options: Max text-completion : 8 . Min text-completion : 0. Not overriding emoji-skype.

Any advice on this?

frissdiegurke

@tedr56
Any errors within js-console?
Does the textarea get wrapped by a div of class textcomplete-wrapper?
If so is it still there if no other plugin that create suggestions like nodebb-plugin-mentions is enabled?

julian

Hmm... @tedr56 -- cleared cache?

Ted

@julian Yes, cleared, as aslways

@frissdiegurke nodebb-plugin-mentions disabled doesn't change anything.

textcomplete-wrapper seems to be there : PasteBin

For Js errors, I couldn't tell. There is :

Connextion with ws://kos.ddns.info/socket.io/1/websocket/J4B98mfcQ2sf70yhY-OF has been interupted during page loading from socket.io.js:2

Using « Mutation Events » is obsolete. Use « MutationObserver » instead. from nodebb.min.js(2)

Using « getPreventDefault() » is obsolete. Use « defaultPrevented » instead. from nodebb.min.js(2)

When accessing composer :

An empty string has been transmited to « getElementById() » coming from nodebb.min.js(1)

Errors are translated. Appearing in french in my Firefox

Ted

I must say the emojis are parsed and decoded well.

Just the suggestions are not working.

julian

@tedr56 Can you try deleting nodebb.min.js from /public/src?

Ted

@julian said:

@tedr56 Can you try deleting nodebb.min.js from /public/src?

And you win again!

Did the trick.

What is this file? I saw it also making trouble in the shoutbox topic.

Is it so insignificant to stay deleted?

julian

Hahah @tedr56 That file is a "minified" version of all of the javascript files. We load perhaps 15-20 separate files (more if there are plugins, and emoji uses 1) normally, and we squish 'em all down into one tiny file in production mode.

Sometimes things go awry and the file doesn't get compiled properly, as seems to be the case here.

Ted

@julian That makes much more sense now. Thanks.

manuel

Can you add a button to insert emoticons in a easy way and to display a list of all available emoji with relative mappings?

frissdiegurke

@manuel Would get a long list... will take a look at this in a few days If you need a list for yourself you may look here

julian

@manuel Perhaps @frissdiegurke can add a "sample" listing (~10 icons) to the "Help" tab in the composer, and a link to the full list? Let me put together a pull request for him...

manuel

@julian +1 but it would be nice to have a toolbar button for lazy users

frissdiegurke

late enough to add some features

Version 0.2.3-3:

Added a help-message.
Added zoom, size configurable via ACP

@julian Anything done with the BONUS POINT of #1409 yet? Will it be dropdown-able?

Planned are:

dropdown to add emoji
remove : from emoji-title, add : to emoji-alt tags therefor copy+paste will not remove them
(figure out better way to exclude code-elements for textcomplete (current way is markdown-support only))
at least deactivate exclusion when markdown-plugin isn't enabled

Any other suggestions?

julian

@frissdiegurke shouldn't be too much work to add the hook in for you to add buttons to the composer toolbar

Let me make a new issue for it... #1444

Xiph

Activating the plugin crashes NodeBB with the following error:
error: Object #<Object> has no method 'persistOnEmpty' TypeError: Object #<Object> has no method 'persistOnEmpty' at resetConfigIfEmpty (C:\nodebb\node_modules\nodebb-plugin-emoji-extended\index.js:1:850) at Object.module.exports.configDefaults (C:\nodebb\node_modules\nodebb-plugin-emoji-extended\index.js:1: 1072) at C:\nodebb\src\plugins.js:347:23 at C:\nodebb\node_modules\async\lib\async.js:111:13 at Array.forEach (native) at _each (C:\nodebb\node_modules\async\lib\async.js:32:24) at Object.async.each (C:\nodebb\node_modules\async\lib\async.js:110:9) at Object.Plugins.fireHook (C:\nodebb\src\plugins.js:345:12) at C:\nodebb\src\plugins.js:400:15 at C:\nodebb\src\plugins.js:100:5 info: [app] Shutdown (SIGTERM/SIGINT) Initialised. info: [app] Database connection closed. info: [app] Shutdown complete.

EDIT: I'd add newlines to the above code block, but that just breaks it =P