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/PaginationSkin.java line 419: > 417: > 418: if (deferredStartup != null) { > 419: deferredStartup.cancel(); We should also `null` the `deferredStartup` here modules/javafx.controls/src/main/java/javafx/scene/control/skin/TableRowSkinBase.java line 147: > 145: public void dispose() { > 146: if (currentTransitions != null) { > 147: currentTransitions.forEach((_, value) -> value.stop()); We should `clear()` the `currentTransitions` here ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2177#discussion_r3402921465 PR Review Comment: https://git.openjdk.org/jfx/pull/2177#discussion_r3402907093
