JetBrains / resharper-angularjs

ReSharper plugin for AngularJS support
Apache License 2.0
134 stars 25 forks source link

Staged completion for ng- attributes #24

Closed citizenmatt closed 10 years ago

citizenmatt commented 10 years ago

I'm concerned that the angular attributes dominate the code completion list, and it's exacerbated by repeating ng-* with data-ng-* and potentially x-ng-* (#14)

This is what I'd like to see:

DerAlbertCom commented 10 years ago

Should be general handled like in Visual Studio 2013 .... R# completion sucks in that case .

citizenmatt commented 10 years ago

Just to check - do you mean that the current R# completion behaviour sucks, or the behaviour I've described above? Would you change what I'm suggesting? I was aiming for something like the VS experience - less noise, but still dead easy to get what you want.

DerAlbertCom commented 10 years ago

R# once ;)

is like the Visual Studio Behavior

DerAlbertCom commented 10 years ago

Look at http://www.asp.net/visual-studio/overview/2013/visual-studio-2013-web-editor-features-html-editor at 1:40

citizenmatt commented 10 years ago

Nice! I'd see the aria-... snippet, but none of the rest. VS is looking pretty good :smile:

blowsie commented 10 years ago

When typing in my html file, I am constantly frustrated by the partial searches in the intellisense.

For example. On a element simply add the attribute style="" just by typing, the second you hit = you will have a data-ng-style="" attribute instead.

This I believe happens because the angular intellisense has some extra words in the search or some sort of partial searching turned on.

This for me is a really bad UX.

angularintellisense

citizenmatt commented 10 years ago

Is this primarily an ordering issue, do you think? If the complete match ("style") was first in the list, would this make it ok?

blowsie commented 10 years ago

Yep, but then it would not be alphabetical like everything else?

citizenmatt commented 10 years ago

I'm not sure on the rules, I'll have to check. Full match should have higher priority, IMO. I'll open a new issue to track.

blowsie commented 10 years ago

@citizenmatt thanks!