search

ergobyte / qookery

Declarative UI Building for Qooxdoo
http://www.qookery.org/
Apache License 2.0
9 stars 5 forks source link

ACE Editor: support autocomplete #9

Closed cboulanger closed 5 years ago

cboulanger commented 5 years ago

https://github.com/ajaxorg/ace/wiki/How-to-enable-Autocomplete-in-the-Ace-editor

gnikolaidis commented 5 years ago

Can you please check that the latest commit on master does the trick? It worked for us quite well.

cboulanger commented 5 years ago

cool! I will try tonight and give you feedback.

gnikolaidis commented 5 years ago

You will have to download ext-language_tools.js into your /lib/ace/ directory and add auto-complete="basic" to the XML element

cboulanger commented 5 years ago

Thanks. That could be a handy addition to the Qookery demo, too. I don't know how well the autocomplete mechanism can handle nested xml, though...

cboulanger commented 5 years ago

No luck yet. This is what I have:

https://github.com/cboulanger/eventrecorder/blob/master/source/class/cboulanger/eventrecorder/UiController.js#L564 https://github.com/cboulanger/eventrecorder/blob/master/source/resource/cboulanger/eventrecorder/forms/editor.xml#L123

No errors, but the console.log in Line 587 is never called. Did I forget something?

cboulanger commented 5 years ago

With #13, it works for me, too.

cboulanger commented 5 years ago

With the latest changes in Master, it works as expected. Thank you very much.

cboulanger commented 5 years ago

It's working very well!

Would you consider releasing the autocomplete support in current master in a patch release so that I can use the contrib system for the travis script and so that I don't have to require users to clone master?

That would be wonderful! Christian

Am 04.03.2019 um 10:31 schrieb George Nikolaidis notifications@github.com:

Can you please check that the latest commit on master does the trick? It worked for us quite well.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ergobyte/qookery/issues/9#issuecomment-469181582, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEmfpcjtwqPvCVZ7WqFb9fmbbXnRW-mks5vTOflgaJpZM4bb58F.

gnikolaidis commented 5 years ago

Sure thing, I just made a 0.7.0 pre-release.