New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Context #91

Closed
dirkk opened this Issue Feb 21, 2014 · 4 comments

Comments

Projects
None yet
3 participants
@dirkk

dirkk commented Feb 21, 2014

I was adding some live templates and noticed that there is no context like XQuery in the list. Would be nice if this could be added as I currently apply it to Other.

By the way, if anyone is interested the live templates are located at https://gist.github.com/dirkk/2fd14096038c1a6c6ea2. Maybe at least some of them could also be included as default in the plugin? Some of them are quite common I would guess

@arnostv

This comment has been minimized.

Show comment
Hide comment
@arnostv

arnostv Feb 21, 2014

Contributor

@dirkk - you can check code from pull request #92, it adds some basic support for live templates, defines one context XQUERY_CODE. Would it work for your templates?
(Note that this only targets Intellij13)

I think it may be useful to define some additional more specific contexts - e.g. some templates can be applicable only within function declaration (declare variable), other only on the top level of xquery module (namespace declaration)...

Contributor

arnostv commented Feb 21, 2014

@dirkk - you can check code from pull request #92, it adds some basic support for live templates, defines one context XQUERY_CODE. Would it work for your templates?
(Note that this only targets Intellij13)

I think it may be useful to define some additional more specific contexts - e.g. some templates can be applicable only within function declaration (declare variable), other only on the top level of xquery module (namespace declaration)...

@ligasgr

This comment has been minimized.

Show comment
Hide comment
@ligasgr

ligasgr Mar 18, 2014

Owner

Hi,
thanks for raising the issue. I'll have a look at the templates.
If I decide to use some of them would you like to contribute them by yourself? Or should I just copy and/or modify them?

Owner

ligasgr commented Mar 18, 2014

Hi,
thanks for raising the issue. I'll have a look at the templates.
If I decide to use some of them would you like to contribute them by yourself? Or should I just copy and/or modify them?

@ligasgr ligasgr added the feature label Mar 19, 2014

@ligasgr

This comment has been minimized.

Show comment
Hide comment
@ligasgr

ligasgr Apr 4, 2014

Owner

I decided to include some as they were, skip some of them, modify some and add some new. I also added you as a contributor to this issue. Additionally I added two sub-contexts (prolog and expression) which should allow for better scoping of live templates.

Owner

ligasgr commented Apr 4, 2014

I decided to include some as they were, skip some of them, modify some and add some new. I also added you as a contributor to this issue. Additionally I added two sub-contexts (prolog and expression) which should allow for better scoping of live templates.

@ligasgr

This comment has been minimized.

Show comment
Hide comment
@ligasgr

ligasgr Apr 4, 2014

Owner

I'm closing the issue (the current theme is implemented). Additional live templates will get added when full text and update facility are implemented.

Owner

ligasgr commented Apr 4, 2014

I'm closing the issue (the current theme is implemented). Additional live templates will get added when full text and update facility are implemented.

@ligasgr ligasgr closed this Apr 4, 2014

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