-
Notifications
You must be signed in to change notification settings - Fork 484
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
8345348: CSS Media Queries #1655
base: master
Are you sure you want to change the base?
Conversation
👋 Welcome back mstrauss! A progress list of the required criteria for merging this PR into |
❗ This change is not yet ready to be integrated. |
* @defaultValue {@link Platform.Preferences#getColorScheme()} | ||
* @since 24 | ||
*/ | ||
private final ObjectProperty<ColorScheme> colorScheme = new MediaProperty<>( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it possible for two different scenes to have different color schemes or any other properties derived from the platform preferences?
would it make more sense to use the platform preferences properties directly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need a way for applications to override the OS preferences. Most applications that offer light and dark themes also allow users to switch between them independently from the OS. The advantage of allowing Scene
to override those properties is that an application can mix and match light and dark themes (so to answer your question: yes, different scenes can have different color schemes).
This might be useful for stylistic or compatibility reasons. For example, an application might use third-party UI components in tool windows that don't work well with a dark theme (maybe even Swing content), so these windows would use a color scheme that is different from the main window.
# Conflicts: # modules/javafx.graphics/src/main/java/com/sun/javafx/application/preferences/PlatformPreferences.java
/reviewers 2 reviewer |
@mstr2 has indicated that a compatibility and specification (CSR) request is needed for this pull request. @mstr2 please create a CSR request for issue JDK-8345348 with the correct fix version. This pull request cannot be integrated until the CSR request is approved. |
Implementation of CSS media queries.
Progress
Issue
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.org/jfx.git pull/1655/head:pull/1655
$ git checkout pull/1655
Update a local copy of the PR:
$ git checkout pull/1655
$ git pull https://git.openjdk.org/jfx.git pull/1655/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 1655
View PR using the GUI difftool:
$ git pr show -t 1655
Using diff file
Download this PR as a diff file:
https://git.openjdk.org/jfx/pull/1655.diff
Using Webrev
Link to Webrev Comment