Tag Helper Rewriting is brittle about razor

[jjonescz]

I do not think the first two examples are broken. The current parsing makes sense to me - if you do not use explicit C# transition @(...), we parse as HTML, i.e., if the attribute value starts with quote ', you cannot use that inside. We should spec it like that.

[jaredpar]

. We should spec it like that.

Generally agree. Essentially within a tag helper and you want C# behavior then use the @(...) transition to be explicit. Otherwise your results may vary due to the issues outlined in the text.

[jjonescz]

Note that .razor component parsing behaves equivalently - e.g., given a Component1 with [Parameter] public object Param { get; set; }, then:

<Component1 Param='"'/> am I a string value?"' />

is parsed as Component1 with Param = '"' plus HTML markup am I a string value?"' /> outside of the component. (And it is a compile time error because " is not a valid C# object value.)

[emperador-ming]

As a user, I would like these cases not to compile or, at the very least, raise a warning recommending an explicit @(...) transition.

Thanks so much for the good work, guys.