Skip to content
This repository has been archived by the owner on Dec 19, 2018. It is now read-only.

TagHelperElementCompletionService does not filter schema HTML elements based on TagOutputHint #1225

Closed
NTaylorMullen opened this issue Apr 14, 2017 · 1 comment
Assignees

Comments

@NTaylorMullen
Copy link
Member

No description provided.

@NTaylorMullen NTaylorMullen added this to the 2.0.0-preview1 milestone Apr 14, 2017
@NTaylorMullen NTaylorMullen self-assigned this Apr 14, 2017
@NTaylorMullen NTaylorMullen changed the title TagHelperElementCompletionService does not filter schema HTML elements when there's a TagHelperPrefix TagHelperElementCompletionService does not filter schema HTML elements based on TagOutputHint Apr 14, 2017
NTaylorMullen added a commit that referenced this issue Apr 14, 2017
- Prior to this if you had a `TagHelper` whos output hint was not in the final form of the completions we'd fall through into a schema check which resulted in a completion being added when it shouldn't. Example: `TagHelper` for `my-tr` that has an output hint of `tr` would end up getting added to the completion list under `body` because `my-tr` was not in the schema and would then be treated like the `environment` `TagHelper` (a new element).
- Added a test to validate that `TagHelper`s with output hints are cross referenced vs. existing completions and do not fall through to a schema check.

#1225
NTaylorMullen added a commit that referenced this issue Apr 14, 2017
- Prior to this if you had a `TagHelper` whos output hint was not in the final form of the completions we'd fall through into a schema check which resulted in a completion being added when it shouldn't. Example: `TagHelper` for `my-tr` that has an output hint of `tr` would end up getting added to the completion list under `body` because `my-tr` was not in the schema and would then be treated like the `environment` `TagHelper` (a new element).
- Added a test to validate that `TagHelper`s with output hints are cross referenced vs. existing completions and do not fall through to a schema check.

#1225
NTaylorMullen added a commit that referenced this issue Apr 14, 2017
- Prior to this if you had a `TagHelper` whos output hint was not in the final form of the completions we'd fall through into a schema check which resulted in a completion being added when it shouldn't. Example: `TagHelper` for `my-tr` that has an output hint of `tr` would end up getting added to the completion list under `body` because `my-tr` was not in the schema and would then be treated like the `environment` `TagHelper` (a new element).
- Added a test to validate that `TagHelper`s with output hints are cross referenced vs. existing completions and do not fall through to a schema check.

#1225
NTaylorMullen added a commit that referenced this issue Apr 14, 2017
- Prior to this if you had a `TagHelper` whos output hint was not in the final form of the completions we'd fall through into a schema check which resulted in a completion being added when it shouldn't. Example: `TagHelper` for `my-tr` that has an output hint of `tr` would end up getting added to the completion list under `body` because `my-tr` was not in the schema and would then be treated like the `environment` `TagHelper` (a new element).
- Added a test to validate that `TagHelper`s with output hints are cross referenced vs. existing completions and do not fall through to a schema check.

#1225
NTaylorMullen added a commit that referenced this issue Apr 14, 2017
- Prior to this if you had a `TagHelper` whos output hint was not in the final form of the completions we'd fall through into a schema check which resulted in a completion being added when it shouldn't. Example: `TagHelper` for `my-tr` that has an output hint of `tr` would end up getting added to the completion list under `body` because `my-tr` was not in the schema and would then be treated like the `environment` `TagHelper` (a new element).
- Added a test to validate that `TagHelper`s with output hints are cross referenced vs. existing completions and do not fall through to a schema check.

#1225
@NTaylorMullen
Copy link
Member Author

d273c6c

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant