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.

Sign up for GitHub

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

Jump to bottom

Allow the user to register the application delegate on macOS and iOS #3758

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Allow the user to register the application delegate on macOS and iOS #3758

wants to merge 4 commits into from
+461 −271

Commits on Jun 24, 2024

  1. Use notifications instead of top-level application delegate methods

    @madsmtm
    madsmtm committed Jun 24, 2024
    Configuration menu
    Copy the full SHA
    b83cfa3 View commit details
    Browse the repository at this point in the history

  2. Don't act as-if the application successfully open all URLs 

    This is a breaking change, although unlikely to matter in practice,
since the return value of `application:didFinishLaunchingWithOptions:`
is seldom used by the system.
    @madsmtm
    madsmtm committed Jun 24, 2024
    Configuration menu
    Copy the full SHA
    4ba3039 View commit details
    Browse the repository at this point in the history

  3. Don't register an application delegate 

    This allows the user to override the application delegate themselves,
which opens several doors for customization that were previously closed.
    @madsmtm
    madsmtm committed Jun 24, 2024
    Configuration menu
    Copy the full SHA
    4a53350 View commit details
    Browse the repository at this point in the history

Commits on Jun 28, 2024

  1. Document the corresponding method names for each notification

    @madsmtm
    madsmtm committed Jun 28, 2024
    Configuration menu
    Copy the full SHA
    fa7a43d View commit details
    Browse the repository at this point in the history