Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-02-14 Thread Kevin Rushforth
On Tue, 13 Feb 2024 13:33:36 GMT, John Hendrikx wrote: > CSR was created I reviewed the CSR. - PR Comment: https://git.openjdk.org/jfx/pull/1340#issuecomment-1943718956

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-02-13 Thread John Hendrikx
On Sat, 10 Feb 2024 15:13:44 GMT, Kevin Rushforth wrote: >>> > need 2 passes, because there already is a method named like that: >>> >>> oh yeah, sorry. `getStyleClassesSet()` ? >> >> That one is still available yes :) > > @hjohn I'm now leaning towards either `getStyleClassNames` or > `getCla

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-02-10 Thread Kevin Rushforth
On Mon, 5 Feb 2024 22:25:36 GMT, John Hendrikx wrote: >>> need 2 passes, because there already is a method named like that: >> >> oh yeah, sorry. >> `getStyleClassesSet()` ? > >> > need 2 passes, because there already is a method named like that: >> >> oh yeah, sorry. `getStyleClassesSet()` ? >

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-02-05 Thread John Hendrikx
On Mon, 5 Feb 2024 22:22:25 GMT, Andy Goryachev wrote: > > need 2 passes, because there already is a method named like that: > > oh yeah, sorry. `getStyleClassesSet()` ? That one is still available yes :) - PR Comment: https://git.openjdk.org/jfx/pull/1340#issuecomment-1928279865

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-02-05 Thread Andy Goryachev
On Mon, 5 Feb 2024 22:14:06 GMT, John Hendrikx wrote: > need 2 passes, because there already is a method named like that: oh yeah, sorry. `getStyleClassesSet()` ? - PR Comment: https://git.openjdk.org/jfx/pull/1340#issuecomment-1928264474

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-02-05 Thread Andy Goryachev
On Fri, 2 Feb 2024 16:00:28 GMT, Kevin Rushforth wrote: > I think `getClasses` is fine. My second choice would be `getClassNames`. Perhaps it might be `getStyleClasses()` to parallel one in `Node`? (I don't know why Node's method is called `getStyleClass` (singular) when it clearly returns mult

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-02-05 Thread John Hendrikx
On Mon, 5 Feb 2024 17:19:35 GMT, Andy Goryachev wrote: > > I think `getClasses` is fine. My second choice would be `getClassNames`. > > Perhaps it might be `getStyleClasses()` to parallel one in `Node`? (I don't > know why Node's method is called `getStyleClass` (singular) when it clearly > re

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-02-02 Thread Kevin Rushforth
On Fri, 2 Feb 2024 19:04:33 GMT, John Hendrikx wrote: >>> My conclusion is that we either need two rounds to get this right (to first >>> free up our first choice getStyleClasses), or we can settle on one of: >>> getStyleClassNames, getStyles, getStyleNames, getClasses, getClassNames. >> >> Le

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-02-02 Thread John Hendrikx
On Fri, 2 Feb 2024 16:00:28 GMT, Kevin Rushforth wrote: > There is one other method used by SceneBuilder's `CssInternal` class, > `CompoundSelector::getSelectors`. I think you need to add a method to the > `Selector` base class that returns a `List` instead of the > `List` returned by the exis

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-02-02 Thread Kevin Rushforth
On Fri, 2 Feb 2024 13:27:50 GMT, John Hendrikx wrote: > My conclusion is that we either need two rounds to get this right (to first > free up our first choice getStyleClasses), or we can settle on one of: > getStyleClassNames, getStyles, getStyleNames, getClasses, getClassNames. Let's settle o

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-02-02 Thread John Hendrikx
On Fri, 19 Jan 2024 16:00:49 GMT, John Hendrikx wrote: >> The SimpleSelector and CompoundSelector classes are public classes in an >> exported package, javafx.css, but they are not intended to be used by >> applications. They are implementation details. They cannot be constructed >> directly a

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-01-24 Thread Kevin Rushforth
On Fri, 19 Jan 2024 16:00:49 GMT, John Hendrikx wrote: >> The SimpleSelector and CompoundSelector classes are public classes in an >> exported package, javafx.css, but they are not intended to be used by >> applications. They are implementation details. They cannot be constructed >> directly a

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-01-19 Thread John Hendrikx
On Fri, 19 Jan 2024 16:00:49 GMT, John Hendrikx wrote: >> The SimpleSelector and CompoundSelector classes are public classes in an >> exported package, javafx.css, but they are not intended to be used by >> applications. They are implementation details. They cannot be constructed >> directly a

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-01-19 Thread Nir Lisker
On Fri, 19 Jan 2024 16:00:49 GMT, John Hendrikx wrote: >> The SimpleSelector and CompoundSelector classes are public classes in an >> exported package, javafx.css, but they are not intended to be used by >> applications. They are implementation details. They cannot be constructed >> directly a

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-01-19 Thread John Hendrikx
On Fri, 19 Jan 2024 17:54:17 GMT, Andy Goryachev wrote: > > Scene Builder releases are always tight to the JavaFX version > > Thank you for clarification! Still, there might be other tools out there that > we don't know of. That's always possible. I couldn't find anything with an internet sea

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-01-19 Thread Jose Pereda
On Fri, 19 Jan 2024 16:00:49 GMT, John Hendrikx wrote: >> The SimpleSelector and CompoundSelector classes are public classes in an >> exported package, javafx.css, but they are not intended to be used by >> applications. They are implementation details. They cannot be constructed >> directly a

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-01-19 Thread Andy Goryachev
On Fri, 19 Jan 2024 17:52:31 GMT, Jose Pereda wrote: > Scene Builder releases are always tight to the JavaFX version Thank you for clarification! Still, there might be other tools out there that we don't know of. - PR Comment: https://git.openjdk.org/jfx/pull/1340#issuecomment-1900

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-01-19 Thread Andy Goryachev
On Fri, 19 Jan 2024 16:00:49 GMT, John Hendrikx wrote: >> The SimpleSelector and CompoundSelector classes are public classes in an >> exported package, javafx.css, but they are not intended to be used by >> applications. They are implementation details. They cannot be constructed >> directly a

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-01-19 Thread John Hendrikx
On Fri, 19 Jan 2024 16:00:49 GMT, John Hendrikx wrote: >> The SimpleSelector and CompoundSelector classes are public classes in an >> exported package, javafx.css, but they are not intended to be used by >> applications. They are implementation details. They cannot be constructed >> directly a

Re: RFR: JDK-8324182 Deprecate for removal SimpleSelector and CompoundSelector classes [v2]

2024-01-19 Thread John Hendrikx
> The SimpleSelector and CompoundSelector classes are public classes in an > exported package, javafx.css, but they are not intended to be used by > applications. They are implementation details. They cannot be constructed > directly and no other JavaFX API accepts or returns a SimpleSelector or