mentions icon indicating copy to clipboard operation
mentions copied to clipboard

autocomplete for usernames is broken

Open propertunist opened this issue 8 years ago • 5 comments

i am not able to get the autocomplete dropdown box to render in any of my sites (1.12.x). if i type a @ symbol followed by some characters i see no dropdown. i have tested with my theme disabled and with/without tinyMCE and the result is the same in all cases.

propertunist avatar Apr 23 '16 20:04 propertunist

i have started looking at the code here and found that in autocomplete.js:

a) the timeout of 500ms that is present to delay the check to find out if tinyMCE is instantiated is too short on my machine. if i increase the delay to 1000ms then the tinyMCE code is run, but if i leave it as 500ms then it is not run.

b) when the tinyMCE code is run, the following line generates an error in the console that says that onKeyUp is not defined:

tinymce.editors[i].onKeyUp.add(function (ed, e) {

propertunist avatar Apr 23 '16 23:04 propertunist

does anyone know if this plugin has been tested with tinyMCE 4+ ?

propertunist avatar Apr 23 '16 23:04 propertunist

i have now upgraded mentions in my fork of it to be compatible with tinyMCE 4+ and elgg 2.1.2. the remaining issues are not related to tinyMCE.

propertunist avatar Jul 01 '16 21:07 propertunist

Can you make a PR?

juho-jaakkola avatar Jul 02 '16 17:07 juho-jaakkola

ok, done

propertunist avatar Jul 02 '16 17:07 propertunist