Skip to content

Conversation

@hebasto
Copy link
Member

@hebasto hebasto commented Nov 17, 2022

MacOS 13 sends a window focus change notification after the main
window has been destroyed but before the QTApplication has been
destroyed. This results in the menu bar receiving a notification
despite it no longer existing. The solution is to pass the main
window as context when subscribing to the notifications. Qt
automatically unsubscribes to notifications if the sender OR
context is destroyed.

Github-Pull: bitcoin-core/gui#680
Rebased-From: 8a5014c
Copy link
Contributor

@jarolrod jarolrod left a comment

Choose a reason for hiding this comment

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

ACK e54a4de

Confirmed the issue by running 23.x on macOS Ventura, this backport fixes the issue.

@DrahtBot
Copy link
Contributor

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK jarolrod

@fanquake fanquake merged commit d14dc8e into bitcoin:23.x Nov 21, 2022
@hebasto hebasto deleted the 221117-23.1-gui-bp branch November 21, 2022 10:11
fanquake added a commit that referenced this pull request Nov 22, 2022
87c2ff8 doc: add release notes for 23.1rc2 (fanquake)
5b2c99e doc: Update manual pages for 23.1rc2 (fanquake)
a02ae07 build: Bump version to 23.1rc2 (fanquake)
a1fcd56 doc: add 23.0 relase notes (fanquake)

Pull request description:

  Bump the version to 23.1rc2.
  Regenerate the man pages.
  Add historical 23.0 release notes.
  Add WIP 23.1 release notes.

  Changes since rc1:
  * #26474
  * #26522

ACKs for top commit:
  hebasto:
    ACK 87c2ff8, I have reviewed the code and it looks OK, I agree it can be merged.
  jarolrod:
    ACK 87c2ff8

Tree-SHA512: f185c33bfe2bf26f9df48f8d1e95ab784fef055bc7465282e2418960d348bd8b35f8fc57808b47250f8536a9def3be2d1dbe8ae309f052a81220925c425108f1
jamesdorfman added a commit to jamesdorfman/elements that referenced this pull request Sep 25, 2023
@bitcoin bitcoin locked and limited conversation to collaborators Nov 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants