On Thu, 13 Nov 2025 21:24:25 GMT, Michael Strauß <[email protected]> wrote:
> The implementation of `Dialog.dialogPane` adds a `ListChangeListener` to > `DialogPane.getButtonTypes()`, and calls `DialogPane.requestLayout()` from > the listener. > > This is not needed, because the `DialogPane.createButtonBar()` method already > specifies that implementers are required to observe changes of > `DialogPane.getButtonTypes()`, and relayout the button bar as needed. > > I've added a test to verify that changing the button types indeed requests a > layout on the `DialogPane`. Despite the questions about the test, the fix seems right - I see no regression. please wait the customary 24 hrs before integrating, in case someone else wants to take a look at it. ------------- Marked as reviewed by angorya (Reviewer). PR Review: https://git.openjdk.org/jfx/pull/1973#pullrequestreview-3462114044 PR Comment: https://git.openjdk.org/jfx/pull/1973#issuecomment-3530148442
