RFR: 8313424: JavaFX controls in the title bar [v47]

Andy Goryachev angorya at openjdk.org
Tue Feb 4 20:56:28 UTC 2025


On Tue, 4 Feb 2025 12:41:24 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:

>> Implementation of [`EXTENDED`](https://gist.github.com/mstr2/0befc541ee7297b6db2865cc5e4dbd09) and `EXTENDED_UTILITY` stage style.
>
> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
> 
>   add "maximized" pseudo-class for custom maximize button

I wonder if the header bar needs to get clipped somehow:

![Screenshot 2025-02-04 at 12 50 23](https://github.com/user-attachments/assets/29f8be40-3b47-4192-a6c5-eeca00b22584)

(also weird padding around buttons in the left content area)

modules/javafx.base/src/test/java/test/util/ReflectionUtils.java line 41:

> 39:      * The field can be declared on any of the object's inherited classes.
> 40:      */
> 41:     @SuppressWarnings("unchecked")

is this annotation still needed (the build gives no warnings when it's removed)?

modules/javafx.graphics/src/main/native-glass/gtk/glass_window.h line 2:

> 1: /*
> 2:  * Copyright (c) 2011, 2024, Oracle and/or its affiliates. All rights reserved.

we probably want to change this to 2025 (and merge the latest master for good measure)

tests/manual/monkey/src/com/oracle/tools/fx/monkey/tools/StageTesterWindow.java line 186:

> 184:         sizeComboBox.valueProperty().subscribe(event -> updateMinHeight.run());
> 185:         headerBar.minSystemHeightProperty().subscribe(event -> updateMinHeight.run());
> 186:         headerBar.setLeading(new Button("✨"));

minor: this is nice, but can we replace this with non-emoji character please?  (messes up the line spacing in Eclipse)

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

PR Comment: https://git.openjdk.org/jfx/pull/1605#issuecomment-2635049491
PR Review Comment: https://git.openjdk.org/jfx/pull/1605#discussion_r1941811731
PR Review Comment: https://git.openjdk.org/jfx/pull/1605#discussion_r1941806528
PR Review Comment: https://git.openjdk.org/jfx/pull/1605#discussion_r1941871727


More information about the openjfx-dev mailing list