Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: #1547: whenWillCloseDo: block is evaluated twice when closing a window #1566

Closed
wants to merge 1 commit into from

Conversation

Ducasse
Copy link
Contributor

@Ducasse Ducasse commented Jun 28, 2024

We decided to remove the closeDiscardingChanges from Spec

@Ducasse
Copy link
Contributor Author

Ducasse commented Jun 28, 2024

This is a potential backport to P12.

@estebanlm
Copy link
Member

please do not merge this, I let a comment in the reported issue.

@MarcusDenker
Copy link
Contributor

What is important, too: please add a better name tan "Fix: #1547" (this looks very bad in the weekly email, too)

@MarcusDenker MarcusDenker changed the title Fix: #1547 Fix: #1547: whenWillCloseDo: block is evaluated twice when closing a window Jul 3, 2024
@Ducasse
Copy link
Contributor Author

Ducasse commented Sep 18, 2024

Right now the code of the book breaks.

@estebanlm
Copy link
Member

Right now the code of the book breaks.

can you point me to the example that breaks ?

@Ducasse
Copy link
Contributor Author

Ducasse commented Sep 19, 2024

@koendehondt Do you remember this is in the window chapter?

@koendehondt
Copy link
Contributor

Right now the code of the book breaks.

can you point me to the example that breaks ?

The example in the description of #1547 exposes the bug.

The code in the book is in section 9.3 "Preventing window close". The corresponding code is in class WindowExamplePresenter2 in the code repo. The class comment links to #1547 to indicate that the example does not work.

@estebanlm
Copy link
Member

This is fixed upstream (I applied same thing)

@estebanlm estebanlm closed this Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants