[nodebb-plugin-emoji-extended] Emoji Extended (reworked)
I've just published the first (except
emoji-extendedxD ) releases of
All of those depend on
Take a look at the ACP settings of
nodebb-plugin-emoji-extendedto see and feel all improvements
The (un)install buttons are currently not working and thus disabled.
Feedback is always welcome.
Here is my current TODO list:
Emoji Static * [s/c] show alert with reason of error when index not found * [s/c] image upload * [s/c] ACP details-handling (deprecate *index.json*) * [c] multiple sets management Emoji Extended * [s/c] enable within signatures (including auto-completion) * [s] allow sets to specify outdated files (e.g. index-file check), indicate within ACP * [s/c] allow user to specify prefix/suffix for auto-completion per set, auto-create suffix (setId) for overwritten emoji * [c] check markdown-alternatives for code-block detection implementation * [s/c] allow alternate image source for modal (e.g. sprites image) * [s/c] ACP page * [c] add sort capability of active sets * [s/c] indicate busy-state on page-load * [s/c] implement [un]install actions * [s/c] show modules that are installed but not enabled within NodeBB (on top of available sets) * [s/c] implement activation action * [s] filter modules for installation that ain't compatible with NodeBB version * [c] (maybe) markdown check rework * [s/c] (maybe) push-update styles on sets-update, append to already in-use styles Emoji One * [s] parse unicode emoji characters New Sets * twemoji * PhantomOpenEmoji * emojisymbols.com
Of course if you want to contribute, feel encouraged to send a PR or even publish your own sets
However I'm not sure in which way NodeBB should/could keep emoji-extended within its dependencies.
If emoji-extended is listed (and activated by default), some unused client-side code gets delivered.
If emoji-extended and non-static set(s) are listed (and activated by default), that/those set(s) would still need to update files by hand via emoji-extended ACP page.
The options that come to my mind:
emoji-extendedand a static set (
emoji-cubicoppis the only one right now); add that set to default settings of
emoji-extendedtherefor it is activated by default.
emoji-extendedand a non-static set (
emoji-onewould be my choice); add that to default settings; call its
./nodebb setup; I could add a callback-based variant for that set if you want to avoid promises within core.
emoji-extendedand a non-static set; add that to default settings; call its
update()function on server start (
static:app.preloadI assume) if no files are present.
emoji-extended; do not enable it by default
- drop emoji from dependencies
The 3rd would break my clear distinction between plugin installation and image files o_O .
I tend to the 5th; Adding the word "smiley" to the description of emoji-extended should ease the search within the plugin-list.
It would be great if NBBPM could handle
nodebb-[...]-properly (or does it already?) for this case.
@PitaJ But only as long all version ranges match a single version, if they don't it communicates with a "silent" nested
nodebb-plugin-emoji-extendedwhich would probably cause much confusion and thus I don't want this The main problem is that NodeBB at some point breaks the distance to NPM by depending on the
node_modulesstructure (of which I cannot come up with an alternative way of course). This makes it impossible to solve the issue with NPM only. It has to be either a
peerDependencywhich will not install the package if it misses, but does not cause other possible trouble or a solution via NodeBB itself. Since NodeBB breaks the distance to NPM it should be responsible for that breakpoint. At least these are my thoughts of it.
dependencywould also disrespect the
nodebb-plugin-emoji-extendedwhich might cause even more trouble.
I think as long NodeBB does not provide a better way, going with
peerDependencyand a notice within the plugins descriptions is the best I can do.
nodebb-plugin-emoji-appleI see in my log this error:
26/2 10:35  - error: [plugins/emoji] nodebb-plugin-emoji-extended is not installed. nodebb-plugin-emoji-apple depends on it.
I do have
nodebb-plugin-emoji-extendedinstalled. To make sure nothing was wrong with the base emoji-extended plugin i removed it and the newly installed emoji-apple plugin, reloaded NodeBB, and re-installed but still receive this warning message.
It seems that the plugin is working though but feel that I should report this as something maybe wrong that I haven't found yet.
@rod I assume you have any
1.0.0installed. I should precise that message o_O
FYIL: to install any
^1.0.0version you currently have to remove it from the package.json file
And if this is the case it's not using the
@rod If you're on master you can install it without any problems, just need to remove that listing within package.json (you'll have to
git reset --hardbefore future
git pullto avoid problems with
If you're on another branch (e.g.
0.9.x) it's probably not working to its full potential. For example I think you'd have to update the composer too in order to get auto-completion...
@ThingBreaker And as to the translations it seems the
Since it's a minor
0.9.xissue and I'm supporting
^1.0.0only, it's not gonna be fixed from my side. Maybe the @administrators want to merge the fix of
0.9.x, but I wouldn't bet on this . I guess it's something one can live with until next major NodeBB release.