Skip to content

CodeMirror extension: addon/hint/Code snippets#7

Open
as3boyan wants to merge 2 commits into
angelozerr:masterfrom
as3boyan:master
Open

CodeMirror extension: addon/hint/Code snippets#7
as3boyan wants to merge 2 commits into
angelozerr:masterfrom
as3boyan:master

Conversation

@as3boyan
Copy link
Copy Markdown

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?

Stop jumping between variables on ${cursor}
@as3boyan as3boyan changed the title CodeMirror externsion: addon/hint/Code snippets CodeMirror extension: addon/hint/Code snippets Mar 24, 2014
@as3boyan
Copy link
Copy Markdown
Author

as3boyan commented May 4, 2014

Indentation issues fixed in 8a25f24.

@as3boyan
Copy link
Copy Markdown
Author

as3boyan commented May 4, 2014

@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
Copy link
Copy Markdown
Author

as3boyan commented May 4, 2014

Never mind, I was able to do this.

@angelozerr
Copy link
Copy Markdown
Owner

@as3boyan
Copy link
Copy Markdown
Author

as3boyan commented May 4, 2014

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
Copy link
Copy Markdown
Author

as3boyan commented May 4, 2014

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants