RFR: 8301302: Platform preferences API [v23]

Michael Strauß mstrauss at openjdk.org
Fri Nov 17 23:46:10 UTC 2023


On Fri, 17 Nov 2023 19:20:32 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Rename Appearance to ColorScheme
>
> modules/javafx.graphics/src/main/java/javafx/application/Platform.java line 578:
> 
>> 576:          * from the perceptual brightness of {@link #backgroundColorProperty() backgroundColor} in relation
>> 577:          * to {@link #foregroundColorProperty() foregroundColor} and defaults to {@link ColorScheme#LIGHT}
>> 578:          * if the platform does not report color preferences.
> 
> Do we want to allow for the possibility of a platform reporting the color scheme directly (rather than specifying that it is always derived from the relative brightness of the foreground and background colors)? I can't think of a need off hand.

I've removed the language that specified how the color scheme is determined, as it is immaterial for read-only platform preferences. This question will become relevant for mutable application preferences, where we will need to decide whether `colorScheme` is automatically derived from foreground and background colors, or whether it will need to be set by application developers _along with_ the foreground and background colors.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1014#discussion_r1397995715


More information about the openjfx-dev mailing list