-
Notifications
You must be signed in to change notification settings - Fork 34.4k
Closed
Labels
bugIssue identified by VS Code Team member as probable bugIssue identified by VS Code Team member as probable bugemmetEmmet related issuesEmmet related issuesverifiedVerification succeededVerification succeeded
Milestone
Description
- VSCode Version: Code - Insiders 1.14.0-insider (fda3ed3, 2017-06-14T05:06:38.644Z)
- OS Version: Darwin x64 16.6.0
- Extensions:
Extension | Author | Version |
---|---|---|
ng-template | Angular | 0.1.3 |
vscode-markdownlint | DavidAnson | 0.9.1 |
code-settings-sync | Shan | 2.8.1 |
sort-lines | Tyriar | 1.3.0 |
Bookmarks | alefragnani | 0.15.1 |
project-manager | alefragnani | 0.18.0 |
vscode-eslint | dbaeumer | 1.2.11 |
vscode-html-css | ecmel | 0.1.7 |
tslint | eg2 | 0.15.0 |
auto-rename-tag | formulahendry | 0.0.12 |
Angular2 | johnpapa | 2.3.3 |
theme-karyfoundation-themes | karyfoundation | 14.0.0 |
HTMLHint | mkaufman | 0.3.3 |
vscode-stylefmt | mrmlnc | 2.4.1 |
angular2-inline | natewallace | 0.0.17 |
vscode-code-outline | patrys | 0.0.6 |
stylelint | shinnn | 0.24.8 |
darcula-extended | smlombardi | 3.3.2 |
slime | smlombardi | 2.6.6 |
theme-tesla | smlombardi | 6.0.0 |
change-case | wmaurer | 1.0.0 |
Steps to Reproduce:
- Take a piece of html like
<select class="form-control focus">
- At the end, after the
class
try to add anid
by typingid
. Intellisense shows 2id
entries. - Take the default.
- You wind up with
<select class="form-control focus" <id></id>>
This is obviously incorrect.
Since AFAIK there is no html <id></id>
tag, why is this even a completion? And, even if you want to preserve that completion as an option, shouldn't Code default to the second Intellisense hint, the correct completion of id=""
?
I would further suggest that tags should not be offered as completions when you're already inside a tag.
Metadata
Metadata
Assignees
Labels
bugIssue identified by VS Code Team member as probable bugIssue identified by VS Code Team member as probable bugemmetEmmet related issuesEmmet related issuesverifiedVerification succeededVerification succeeded