On Fri, 11 Apr 2025 17:40:09 GMT, Michael Strauß <mstra...@openjdk.org> wrote:

>> Implementation of [CSS media 
>> queries](https://gist.github.com/mstr2/cbb93bff03e073ec0c32aac317b22de7).
>
> Michael Strauß has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains 17 commits:
> 
>  - Merge branch 'master' into feature/media-queries
>  - add more tests
>  - Merge branch 'master' into feature/media-queries
>  - small refactor
>  - make media-query types internal
>  - add documentation
>  - add missing javadoc tag
>  - revert import
>  - move scene preferences to Scene.Preferences interface
>  - revert
>  - ... and 7 more: https://git.openjdk.org/jfx/compare/8a61dd2b...3c5bbb84

Partial review: the test cases are pretty comprehensive, thanks!

modules/javafx.graphics/src/test/java/test/javafx/scene/Scene_preferences_Test.java
 line 86:

> 84:             new TestRun("prefers-reduced-data",
> 85:                     prefs -> prefs.setReducedData(false),
> 86:                     prefs -> prefs.setReducedData(true))

it's probably an IDE artifact, but indentation is all over the place.

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

PR Review: https://git.openjdk.org/jfx/pull/1655#pullrequestreview-2761718408
PR Review Comment: https://git.openjdk.org/jfx/pull/1655#discussion_r2040384310

Reply via email to