Added a note about prop modifiers on render function examples #1712
Conversation
✅ Deploy Preview for vuejs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
src/api/built-in-directives.md
Outdated
| @@ -328,6 +328,15 @@ Dynamically bind one or more attributes, or a component prop to an expression. | |||
| <div .someProperty="someObject"></div> | |||
| ``` | |||
|
|
|||
| The `.attr` modifier has a shorthand `^`: | |||
There was a problem hiding this comment.
I don't think the ^ template shorthand should be documented.
It was documented previously but we reverted it because it doesn't work. See #1658. After that PR was merged I confirmed with Evan that this is intentional and it shouldn't be included in the docs.
To be clear, it is specifically the template shorthand that doesn't work. Using it as a prefix in a render function works fine.
There was a problem hiding this comment.
@skirtles-code thanks! Removed the added part from directives doc
src/guide/extras/render-function.md
Outdated
| @@ -39,6 +39,9 @@ h('div', { id: 'foo' }) | |||
| // Vue automatically picks the right way to assign it | |||
| h('div', { class: 'bar', innerHTML: 'hello' }) | |||
|
|
|||
| // props modifiers such as .prop and .attr can be added via shorthands | |||
There was a problem hiding this comment.
I think this explanation will need tweaking, as we can't document ^ as a shorthand.
There was a problem hiding this comment.
Updated the explanation
Description of Problem
See #1649
Proposed Solution
Add a shorthand for
.attrin the directives guide and a note on how to use modifiers in the render functionClose #1649