RFR: 8301302: Platform preferences API [v47]

Jose Pereda jpereda at openjdk.org
Thu Dec 7 18:26:55 UTC 2023


On Thu, 7 Dec 2023 17:21:05 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:

>> Please read [this document](https://gist.github.com/mstr2/9f46f92c98d3c86aa6a0b4224a9a6548) for an introduction to the Platform Preferences API, and how it interacts with the proposed style theme and stage appearance features.
>
> Michael Strauß has updated the pull request incrementally with three additional commits since the last revision:
> 
>  - removed unused import
>  - javadoc
>  - query resource bundles for high-contrast schemes only on Windows

Manual test on macOS and Windows looks good now. I've left one comment

modules/javafx.graphics/src/main/java/com/sun/javafx/application/WindowsHighContrastScheme.java line 43:

> 41:  * The high contrast feature may not be available on all platforms.
> 42:  */
> 43: enum WindowsHighContrastScheme {

I'd rather keep the old name: `HighContrastScheme`: It is in a non-platform specific package, and it applies to all platforms (even only with `NONE`).
You could add a comment about `HIGH_CONTRAST_*` enum constants being only available on Windows.

-------------

PR Review: https://git.openjdk.org/jfx/pull/1014#pullrequestreview-1770790578
PR Review Comment: https://git.openjdk.org/jfx/pull/1014#discussion_r1419429539


More information about the openjfx-dev mailing list