-
Notifications
You must be signed in to change notification settings - Fork 27.4k
doc(ngIf): mention multiElement in documentation #11104
Conversation
@@ -32,6 +32,9 @@ | |||
* Additionally, you can provide animations via the `ngAnimate` module to animate the `enter` | |||
* and `leave` effects. | |||
* | |||
* You can use ng-if across few dom elements. First element needs `ng-if-start="expresion"`, and the | |||
* last one `ng-if-end`. Those two and all elements between will be hide/appear together. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will hide (without be)
I am not sure this should be documentd separately on every But it might be a good idea to introduce a |
1f42bf8
to
e6e6498
Compare
while |
It could be a minimal template pointing to the more detailed explanation in |
I agree with @gkalpak, an annotation seems like a good idea. |
Do I get it right that this annotation should be implemented in https://github.com/angular/dgeni-packages, by a new module similar to |
Yes, that is correct! Though I'd like to know what @petebacondarwin thinks of introducing a tag for |
Sounds good to me |
…t multiElement See angular/angular.js#11104 Closes #109
The change has landed on dgeni-packages master, just need to run a release then we will need a docs PR to update the directives that support multi-element |
The only problem left is that |
Cool! Later I've realized that multiElement can be the better shown as part of Usage section as: <ANY
ng-repeat-start="">
</ANY>
...
<ANY
ng-repeat-end>
</ANY> But with directive in place, this can be changed later. |
@marcin-wosinek - nice idea. If you fancied trying to do a PR with that change that would be cool |
No description provided.