[jfx20] RFR: 8290863: Update the documentation of Virtualized controls to include the best practice of not using Nodes directly in the item list

Andy Goryachev angorya at openjdk.org
Fri Jan 13 16:45:28 UTC 2023


On Fri, 13 Jan 2023 12:32:53 GMT, Ajit Ghaisas <aghaisas at openjdk.org> wrote:

> This PR adds a warning about inserting Nodes directly into the virtualized containers such as ListView, TreeView, TableView and TreeTableView. It also adds code snippets showing the recommended pattern of using a custom cell factory for each of the virtualized control.

modules/javafx.controls/src/main/java/javafx/scene/control/ListView.java line 196:

> 194:  * <p> This example has an anonymous custom {@code ListCell} class in the custom cell factory.
> 195:  * Note that the Rectangle (Node) object needs to be created in the custom {@code ListCell} class
> 196:  * or in it's constructor and updated/used in it's updateItem method.

Do we *really* need a code sample to emphasize the fact that Nodes should not be a part of the data model?  
Would it be better to create a bulleted list which briefly, but succinctly describes the rules that should be followed by the developer?
Having said that, there is a certain benefit of having a code sample as it simplifies writing code via copy-paste.

modules/javafx.controls/src/main/java/javafx/scene/control/TreeTableView.java line 295:

> 293:  * given cell and update them on demand using the data stored in the item for that cell.
> 294:  *
> 295:  * <p>For example, rather than use the following code:

perhaps instead of dedicating space to incorrect code, we should provide a correct code block only, emphasizing the fact that the data model must not contain Nodes, in text?

-------------

PR: https://git.openjdk.org/jfx/pull/995


More information about the openjfx-dev mailing list