Vscode-ng-language-service: autocomplete on click should suggest (click)

Created on 30 Nov 2018  路  7Comments  路  Source: angular/vscode-ng-language-service

When i want to add (click) attribute with click, there is no suggestion:
image
However, it will suggest when beginning with (:
image

feature

Most helpful comment

I am in favor of the approach in https://github.com/angular/angular/pull/34445.
The main reason is because the suggestion list is _overly broad_ with the original implementation.
Take the following example:

<div |></div>

where | is the cursor position.
At this point, we are asking the editor, what are the possible candidates here?
The result is a long list of items (html properties/attributes/events with or without binding), and such long list hurts discoverability.

Since this issue was created before the autocomplete behavior was revamped, I suggest we continue the discussion in https://github.com/angular/vscode-ng-language-service/issues/521.

All 7 comments

It should also suggest *ngif when type ngi, with latest version I have to type *ngi

I think this was an intentional change with https://github.com/angular/angular/pull/34445.

cc @kyliau @andrius-pra @ivanwonder thoughts on supporting or closing this?

I am in favor of the approach in https://github.com/angular/angular/pull/34445.
The main reason is because the suggestion list is _overly broad_ with the original implementation.
Take the following example:

<div |></div>

where | is the cursor position.
At this point, we are asking the editor, what are the possible candidates here?
The result is a long list of items (html properties/attributes/events with or without binding), and such long list hurts discoverability.

Since this issue was created before the autocomplete behavior was revamped, I suggest we continue the discussion in https://github.com/angular/vscode-ng-language-service/issues/521.

I'm also prefer the current behavior. It's easier to write [, [( or ( character and choose relevant value from small list of suggestions that valid in your context.

I'm also prefer the current behavior. It's easier to write [, [( or ( character and choose relevant value from small list of suggestions that valid in your context.

Agree.
and I think it's also good to provide bind-, on-, etc on <div |></div>.

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_This action has been performed automatically by a bot._

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Liero picture Liero  路  3Comments

yjaaidi picture yjaaidi  路  5Comments

fxck picture fxck  路  6Comments

frederikschubert picture frederikschubert  路  3Comments

seangwright picture seangwright  路  4Comments