New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: provides collection based append/remove methods for Node #14204
feat: provides collection based append/remove methods for Node #14204
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
} | ||
|
||
/** | ||
* Removes the given child element(s). |
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.
Not exactly sure, but I guess now that these new APIs are accepting a collection it is fine to just use the plural without parenthesis:
* Removes the given child element(s). | |
* Removes the given child elements. |
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.
Collection may contain only one element, so probably that's fine to say "element(s)"
This ticket/PR has been released with Vaadin 23.2.0.beta1 and is also targeting the upcoming stable 23.2.0 version. |
Description
Added Collection based appendChild / appendVirtualChild / insertChild / removeChild / removeVirtualChild methods for
com.vaadin.flow.dom.Node
Fixes #13599
Type of change
Checklist
I have added tests to ensure my change is effective and works as intended.already covered with testsAdditional for
Feature
type of change