BaselineOffset of StackPane is inconsistent

2023-01-24 Thread Scott Palmer
I'm seeing something odd with the alignment of content in a TextFlow and I'm not sure if I'm doing something wrong, or if there is a bug there. The getBaselineOffset() method of the StackPane is returning inconsistent values. If I sub-class it to force a constant offset, everything works. Since th

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

2023-01-24 Thread Nir Lisker
On Tue, 24 Jan 2023 09:59:48 GMT, Ajit Ghaisas wrote: >> modules/javafx.controls/src/main/java/javafx/scene/control/ListView.java >> line 152: >> >>> 150: * Warning: Nodes should not be inserted directly into the items >>> list >>> 151: * ListView allows for the items list to contain element

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

2023-01-24 Thread Nir Lisker
On Tue, 24 Jan 2023 10:35:36 GMT, Ajit Ghaisas wrote: > > > > @nlisker, Can you please file an issue explaining how exactly you would like > `ListView` documentation to be restructured? Yes, I'll probably work on it during RDP 2. It's not the only control I want to go over. - P

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

2023-01-24 Thread Andy Goryachev
On Tue, 24 Jan 2023 10:40:36 GMT, Ajit Ghaisas 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 >> fac

Re: RFR: Merge jfx20 [v2]

2023-01-24 Thread Kevin Rushforth
> Merge `jfx20` branch into `master`. Kevin Rushforth has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:

Integrated: Merge jfx20

2023-01-24 Thread Kevin Rushforth
On Tue, 24 Jan 2023 15:01:57 GMT, Kevin Rushforth wrote: > Merge `jfx20` branch into `master`. This pull request has now been integrated. Changeset: e710b558 Author:Kevin Rushforth URL: https://git.openjdk.org/jfx/commit/e710b55812fbddf45e3e8069e91dd646ae397da8 Stats: 25 lines i

RFR: Merge jfx20

2023-01-24 Thread Kevin Rushforth
Merge `jfx20` branch into `master`. - Commit messages: - Merge jfx20 - 8300664: Missing copyright header in ConditionalBinding.java file The merge commit only contains trivial merges, so no merge-specific webrevs have been generated. Changes: https://git.openjdk.org/jfx/pull/1010

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

2023-01-24 Thread Kevin Rushforth
On Tue, 24 Jan 2023 10:40:36 GMT, Ajit Ghaisas 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 >> fac

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

2023-01-24 Thread Ajit Ghaisas
On Fri, 20 Jan 2023 11:16:04 GMT, Ajit Ghaisas 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 >> fac

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

2023-01-24 Thread Ajit Ghaisas
> 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. Ajit Ghaisas has

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

2023-01-24 Thread Ajit Ghaisas
On Fri, 20 Jan 2023 12:39:03 GMT, Kevin Rushforth wrote: >> Ajit Ghaisas has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Additional review fixes > > modules/javafx.controls/src/main/java/javafx/scene/control/ComboBox.java line > 147: >

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

2023-01-24 Thread Ajit Ghaisas
On Thu, 19 Jan 2023 05:24:06 GMT, Nir Lisker wrote: >> Ajit Ghaisas has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Remove extra spaces > > modules/javafx.controls/src/main/java/javafx/scene/control/ListView.java line > 152: > >> 150:

Re: [External] : Re: Style themes API

2023-01-24 Thread Michael Strauß
Hi Andy, see my comments below: On Mon, Jan 23, 2023 at 10:33 PM Andy Goryachev wrote: > > Let me backtrack a bit and ask a very specific question: what is the problem > (or problems) you are trying to solve? > I would like to understand if the problem exists (more on that below), does a > wo