Conversation
There was a problem hiding this comment.
this file isn't much different from Edit.cshtml but I only use a user-specified HTML attributes (that helper would normally generate) here. suspect "class" attribute would be merged but all other user-specified HTML attributes would just win.
There was a problem hiding this comment.
- at least initially, input tag helper will ignore the "format" bound attribute if "type" is checkbox, hidden, password, or radio.
type="radio"(not in this sample) needs a value to compare to the expression and the "value" HTML attribute Just Works(tm). so under the covers, that may be the only case where MVC tag helpers useTagHelperContext.AllAttributes
|
Should showcase |
|
Would be nice to see a _Layout with a few anchor tag helpers (to show they can be used there). They're perfect for navigation. |
There was a problem hiding this comment.
I think at one point we discussed adding this as a parameter to <form>, something like:
<form addAntiForgeryToken="true">I could just be misremembering though. /cc @DamianEdwards
|
🐃 |
There was a problem hiding this comment.
since <label/> already has a "for" attribute, we should probably use another name for the <label/> / <input/> / <select/> / <textarea/> expression bound attribute. @DamianEdwards any suggestions?
👌
👎 already using
what does the water buffalo mean? |
|
Buffalo is my version of ⌚ , trying to be a trend setter |
. I was thinking a 8b68658 to
ce49b77
Compare
58c3268 to
9f0d7ad
Compare
|
Rebased and updated @yishaigalatzer @Eilon @DamianEdwards please let @NTaylorMullen and I know if the sample and its comments misses anything from our meeting. |
There was a problem hiding this comment.
"items" attribute needn't be so ugly. I just didn't switch from User to UserViewModel in this sample.
d948242 to
b68fae9
Compare
9f0d7ad to
753c3ca
Compare
…scussion - lots of new comments in Create.cshtml
753c3ca to
82cce78
Compare
|
|
Add Tag Helper sample, #1242