Skip to content

Conversation

@neilcsmith-net
Copy link
Member

Attempt to fix issue #6290 in new DialogDisplayerImpl behaviour introduced in #5989 and #6216.

Possible alternative to reverting in #6291

@neilcsmith-net neilcsmith-net added do not merge Don't merge this PR, it is not ready or just demonstration purposes. Platform [ci] enable platform tests (platform/*) ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) labels Aug 3, 2023
@neilcsmith-net neilcsmith-net added this to the NB19 milestone Aug 3, 2023
@neilcsmith-net
Copy link
Member Author

Force pushed (again) because of failing test. Actually realised the test had been updated as part of #5989 and the update here behaves like the previous NetBeans code did. Putting back the old test line.

Copy link
Contributor

@errael errael left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still don't understand the whole leaf thing, but since I haven't tried I don't feel to bad. This certainly seems to avoid parenting to a Window. I'm good with it (except for the initBound()).

Took a news break while reviewing, came across https://www.cnn.com/videos/us/2023/08/02/boston-police-officer-falls-down-slide-cprog-mh-orig.cnn which somehow resonated.

@mbien
Copy link
Member

mbien commented Aug 3, 2023

tested the last dev build of this PR and it seems to be working fine - without exceptions in the log.

I did also notice that the save/discard dialog which pops up when #4792 is used is now centered on the main window, which is better since it used the popup as parent in NB 18 believe.

@neilcsmith-net neilcsmith-net removed the do not merge Don't merge this PR, it is not ready or just demonstration purposes. label Aug 4, 2023
@neilcsmith-net
Copy link
Member Author

Thanks @mbien Also confirmation from the bug reporter on the dev build. Let's merge and test in rc4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) Platform [ci] enable platform tests (platform/*)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants