On Thu, 4 Jun 2026 22:38:39 GMT, Michael Strauß <[email protected]> wrote:
>> Several JavaFX controls use animations to convey state changes, some of >> which are controllable by developers (for example: `TitledPane.animated` or >> `Chart.animated`). However, none of those controls respect the >> `reducedMotion` preference that was introduced with >> [JDK-8341514](https://bugs.openjdk.org/browse/JDK-8341514). >> >> This enhancement changes the behavior of the following control skins to take >> the `Scene.Preferences.reducedMotion` preference into account when >> determining whether to animate a state change: >> >> * `TableRowSkinBase` >> * `TitledPaneSkin` >> * `TabPaneSkin` >> * `PaginationSkin` >> * Charts >> >> For controls whose animation can be toggled by developers, it is useful to >> think of the `animated` property as the application choice, while >> `reducedMotion` is the user choice. >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Michael Strauß has updated the pull request incrementally with one additional > commit since the last revision: > > review comments modules/javafx.controls/src/main/java/javafx/scene/control/skin/TitledPaneSkin.java line 407: > 405: TitledPane skinnable = getSkinnable(); > 406: return skinnable != null > 407: && Platform.isFxApplicationThread() Here and in `TabPaneSkin`, we are checking for `Platform.isFxApplicationThread()`. The others do not. Probably okay, but I thought I would bring it up here. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2177#discussion_r3402870827
