Skip to content
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

Auto-completion needs to work #7

Closed
duemir opened this issue May 16, 2020 · 4 comments
Closed

Auto-completion needs to work #7

duemir opened this issue May 16, 2020 · 4 comments

Comments

@duemir
Copy link
Member

duemir commented May 16, 2020

  • Auto-completion needs to work
@duemir
Copy link
Member Author

duemir commented May 16, 2020

This is a task from Readme.txt. Might be already fixed need to double-check.

@duemir
Copy link
Member Author

duemir commented Sep 13, 2021

Needs a better definition. Some autocompletion definitely work but IntelliJ definitely have no idea about HTML in between Jelly tags (#40)

@Zastai
Copy link

Zastai commented Dec 25, 2021

Aside from HTML there is also an issue completing namespaced tags.

Given a Jelly containing

<?jelly escape-by-default='true'?>
<j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form">

  <f:entry title="Foo" field="bar">
    <f:select/>
  </f:entry>

</j:jelly>

go to the <f:select/> and remove the select leaving just <f:/>. Now press Ctrl-Space with the caret after the :.
This will show a nice list of completions, including the f: prefix - but when selecting one, it will insert that whole entry at the caret, duplicating the f:.

If you start with f:se before pressing Ctrl-Space, then it replaces the se, still duplicating the f:. So there might be some "replace word around point" logic involved, with : not considered a word constituent.

@timja
Copy link
Member

timja commented Nov 11, 2024

Aside from HTML there is also an issue completing namespaced tags.

Given a Jelly containing

<j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form">

<f:entry title="Foo" field="bar">
<f:select/>
</f:entry>

</j:jelly>
go to the <f:select/> and remove the select leaving just <f:/>. Now press Ctrl-Space with the caret after the :. This will show a nice list of completions, including the f: prefix - but when selecting one, it will insert that whole entry at the caret, duplicating the f:.

If you start with f:se before pressing Ctrl-Space, then it replaces the se, still duplicating the f:. So there might be some "replace word around point" logic involved, with : not considered a word constituent.

Created as #190 without seeing this comment


This task isn't clear and actionable so I don't think there's value in keeping it open

@timja timja closed this as not planned Won't fix, can't repro, duplicate, stale Nov 11, 2024
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

No branches or pull requests

3 participants