Skip to content
This repository has been archived by the owner on Feb 16, 2024. It is now read-only.

Application.quitting instead of MonoBehaviour.OnApplicationQuit #463

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

Application.quitting instead of MonoBehaviour.OnApplicationQuit #463

wants to merge 2 commits into from

Commits on Jun 24, 2020

  1. Application.quitting instead of MonoBehaviour.OnApplicationQuit 

    `MonoBehaviour.OnApplicationQuit` is called before `Application.wantsToQuit`
which leads to undesired behavior. `Application.wantsToQuit` can be used to
interrupt a quit process, so you can show the user a window ("do you really want to quit?").
If you cancel the quit, but `MonoBehaviour.OnApplicationQuit` executes anyways, stuff breaks. 
I am aware this changes current behavior and is dealing with inconsistencies in the Unity API
    @Cluster7ck
    Cluster7ck committed Jun 24, 2020
    Configuration menu
    Copy the full SHA
    0026364 View commit details
    Browse the repository at this point in the history

  2. Merge pull request #1 from Cluster7ck/application-quit-patch 

    Application.quitting instead of MonoBehaviour.OnApplicationQuit
    @Cluster7ck
    Cluster7ck committed Jun 24, 2020
    Configuration menu
    Copy the full SHA
    833edd1 View commit details
    Browse the repository at this point in the history