RFR: 8370446: Support dialogs with StageStyle.EXTENDED [v2]

Michael Strauß mstrauss at openjdk.org
Fri Nov 7 01:54:59 UTC 2025


On Thu, 6 Nov 2025 20:54:00 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   move HeaderBar to DialogPane
>
> modules/javafx.controls/src/main/java/javafx/scene/control/Dialog.java line 467:
> 
>> 465:      * Specifies the style for this dialog. This must be done prior to making the dialog visible.
>> 466:      * <p>
>> 467:      * Note that a dialog with the {@link StageStyle#EXTENDED} style must also specify a {@link HeaderBar} with
> 
> must or should?  meaning, is it a requirement, or just a warning?
> if the latter, it might be better to say something like "the dialog will not be draggable if the header bar is not set"

You're right, it's more like a warning. I've changed it to "should".

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1943#discussion_r2501422702


More information about the openjfx-dev mailing list