nlisker

Does the original FXML file need to be modified to be suitable for this change of the class?

I tested it and it works well. Selection is easy and the scrolling works. One thing I noticed is that the acceleration between scrolls is not preserved (doing a scroll...

@hjohn I was confused as well, but the [`TableView` docs](https://openjfx.io/javadoc/22/javafx.controls/javafx/scene/control/TableView.html#sorting-heading) say: > Starting with JavaFX 8.0 (and the introduction of [SortedList](https://openjfx.io/javadoc/22/javafx.base/javafx/collections/transformation/SortedList.html)), it is now possible to have the collection return...

Your example is only applicable to `TableView`. I don't see how you can add and remove sort orders to a single `SortedList`. What you can do is modify its `Comparator`,...

> Is `mtl` a good name for the `prism.order` property? Should it be `metal`? "mtl" is sometimes used a short for "material", which has overlapping context here. I'd prefer the...

I thought about doing this some years ago for `Shape3D`, `Material` and `LightBase` when I was adding light types. `Shape3D` states: > An application should not extend the Shape3D class...

The documentation for `Node` has the line: > An application should not extend the Node class directly. Doing so may lead to an UnsupportedOperationException being thrown. Just above [StringID](https://openjfx.io/javadoc/22/javafx.graphics/javafx/scene/Node.html#StringID).

I forgot to mention `Camera` in my reply. Same as the 3D classes: the user can't do it if they try, so it will fail before compilation. I don't mind...

If this goes forward, please seal `Material` as well.

> > If this goes forward, please seal `Material` as well. > > If we seal `Material`, we should probably also seal `Paint`. What do you think? I don't see...