On Mon, 5 Feb 2024 17:19:35 GMT, Andy Goryachev <ango...@openjdk.org> 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 multiple Strings.)
The problem with that choice is that it would need 2 passes, because there
already is a method named like that:
List<String> getStyleClasses()
We'd first need to get rid of it, then replace it with `Set<String>
getStyleClasses()`.
Keeping it as a `List` I would not recommend, since selectors don't keep
duplicates, and don't keep the order (they never did).
-------------
PR Comment: https://git.openjdk.org/jfx/pull/1340#issuecomment-1928218068
