[GH-ISSUE #191] componentWillUnmount is not called #124

Closed
opened 2026-05-05 11:46:11 -06:00 by gitea-mirror · 2 comments
Owner

Originally created by @unlight on GitHub (Dec 30, 2018).
Original GitHub issue: https://github.com/kusti8/proton-native/issues/191

I just written simple hello world application using proton native.
And I'm wondering why componentWillUnmount is not called, when:

  • I'm doing Quit
  • Closing main window

Is it by design?
I've pasted console.log in these methods.
componentDidMount is working fine.

Win10 Node v10.12.0
`-- proton-native@1.1.10
  `-- libui-node@0.2.1
Originally created by @unlight on GitHub (Dec 30, 2018). Original GitHub issue: https://github.com/kusti8/proton-native/issues/191 I just written simple hello world application using proton native. And I'm wondering why `componentWillUnmount` is not called, when: - I'm doing `Quit` - Closing main window Is it by design? I've pasted console.log in these methods. `componentDidMount` is working fine. ``` Win10 Node v10.12.0 `-- proton-native@1.1.10 `-- libui-node@0.2.1 ```
Author
Owner

@kusti8 commented on GitHub (Feb 1, 2019):

When the main window is closed, then the libui loop is immediately stopped and as a result everything stops. There are callbacks, such as onShouldQuit for menu and onClose for window close. These should do the same things and be more clear than componentWillUnmount.

<!-- gh-comment-id:459863476 --> @kusti8 commented on GitHub (Feb 1, 2019): When the main window is closed, then the libui loop is immediately stopped and as a result everything stops. There are callbacks, such as [onShouldQuit](https://proton-native.js.org/#/component_APIs/app?id=onshouldquit) for menu and [onClose](https://proton-native.js.org/#/component_APIs/window?id=onclose) for window close. These should do the same things and be more clear than componentWillUnmount.
Author
Owner

@kusti8 commented on GitHub (Jan 19, 2020):

Proton Native V2 is now released! If the issue still occurs in the new update, please open a new issue.

<!-- gh-comment-id:576029354 --> @kusti8 commented on GitHub (Jan 19, 2020): Proton Native V2 is now released! If the issue still occurs in the new update, please open a new issue.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/proton-native#124
No description provided.