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

Michael Strauß mstrauss at openjdk.org
Tue Feb 18 20:07:08 UTC 2025


On Tue, 18 Feb 2025 18:26:22 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> This class is not API, so it doesn't matter here. However, it matters in `HeaderBar.leftSystemInset/rightSystemInset`, which are both of type `Dimension2D`. The system insets are not arbitrary rectangles, they are always aligned with the top-left and top-right edges of the window. As such, the only necessary information is their spatial extent, which is best represented as a `Dimension2D`.
>> I'm not sure what you mean by asymmetrical padding. Can you elaborate?
>
> The word "inset" confused me.  Could you use some other word here, if it is referring to a dimension?  Like maybe explain where exactly these are used?

Are we talking about the specification of the system-reserved insets in `HeaderBar`? Or do you suggest to link from `HeaderButtonMetrics` to `HeaderBar`?


    /**
     * Describes the size of the left system-reserved inset, which is an area reserved for the iconify, maximize,
     * and close window buttons. If there are no window buttons on the left side of the window, the returned area
     * is an empty {@code Dimension2D}.
     * <p>
     * Note that the left system inset refers to the left side of the window, independent of layout orientation.
     */
    private final ReadOnlyObjectWrapper<Dimension2D> leftSystemInset =

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1605#discussion_r1960518088


More information about the openjfx-dev mailing list