(fix) don't trigger html completions inside moustache tags

[dummdidumm]

Fixes #89

@jasonlyu123 does this seem like a reasonable fix for you?

[jasonlyu123]

I think it could be moved to HTMLPlugin's doTagComplete. Also, there's also another thing that could trigger this tag close.

<input type="checkbox" checked={ a > b } />

Maybe using a solution similar to what you did to find last open tag to find if there is any unended { before > . And just play it safe and disabling the auto close even if { or } is possibly in a string literal.

[dummdidumm]

You are right with both things. Will do, thanks!

[jasonlyu123]

Find the advantage of doing it in the HtmlPlugin. we could utilize HTML language service's AST to find where the current node start and also get the attribute of the current tag.

const node = html.findNodeAt(offsetAt(position, document.getText()));

I tried this attribute :

<Foo a={a >} />

and it gets parsed as

{
   a:"{a"
}

[dummdidumm]

Good find! I will try to add it to the logic.

[dummdidumm]

I did add your suggestions, works really well. Thanks again!