search

angelozerr / CodeMirror-XQuery

XQuery Editor based on CodeMirror
37 stars 14 forks source link

CodeMirror extension: addon/hint/Code snippets #7

Open as3boyan opened 10 years ago

as3boyan commented 10 years ago

Stop jumping between variables on ${cursor}

Should do the job. I also modified additional files, like templates and I removed unused code, to make it easier for me to use.

The reason I had to put it in another repository is to make it work for Haxe mode in my IDE with minimal additional code. (I decided to stop using Tern in my IDE, because it's too complicated for me, and has too much JavaScript specific things, it's much easier to just use CM addons for it).

In your CodeMirror-XQuery demo, indentation works fine, in my Haxe mode, it's doesn't take in account it. Maybe you know how to fix it?

as3boyan commented 10 years ago

Indentation issues fixed in 8a25f24.

as3boyan commented 10 years ago

@angelozerr Hi, Angelo, could you please tell me how you managed to show small images in completion dialog (I don't need much detail, just want to know which additional changes I need to make it work for CodeMirror autocomplete demo for instance)?

as3boyan commented 10 years ago

Never mind, I was able to do this.

angelozerr commented 10 years ago

If I remember, I hd managed that CSS https://github.com/angelozerr/CodeMirror-XQuery/blob/master/codemirror-javascript/addon/hint/tern/tern-extension.css

as3boyan commented 10 years ago

Thanks. I was able to find a CSS for template here(I am using CM without Tern) https://github.com/angelozerr/CodeMirror-XQuery/blob/master/codemirror-extension/addon/hint/templates-hint.css#L15

I am using your show-context-info and snippets addons in my IDE, they work really great, just one minor thing - show-context-info don't take in account scroll, but that should be easy to implement.

Similar to show it's implemented in show-hints https://github.com/marijnh/CodeMirror/blob/master/addon/hint/show-hint.js#L248

Probably I can implement this on my own.

as3boyan commented 10 years ago

Also I think this line can be removed https://github.com/angelozerr/CodeMirror-XQuery/blob/master/codemirror-extension/addon/hint/templates-hint.js#L262.

It's just creates additional reference to same completion object. I can submit pull requests if you want.