[Linux] [Tiling WM] Close button doesn't work.

[jankaifer]

I confirm (by marking "x" in the [ ] below: [x]):

• This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/.
• This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
• I have read contributing guidelines.

---

Summary
Clicking the close button won't close the window. It might minimize the window on other platforms. But many tiling WMs do not support minimizing (i3 for example). So it just ignores clicking the close button.

Environment

• Operating System: NixOS (Linux) + i3-gaps
• Mattermost Desktop App version: 4.5.2
• Mattermost Server version: 5.32.1

Steps to reproduce
Click the close button.
Press Ctrl+W.
Click option in menu Window > Close.
Make my WM send a signal through the WM_DELETE protocol.

Expected behavior
The application will close.

Observed behavior
It won't close.
I need to send it SIGTERM, SIGKILL, or use xkill.

Possible fixes
Handle close events properly.

[jankaifer]

The issue is here:

desktop/src/main/mainWindow.js

Lines 121 to 124 in 3182a9d

	if (global.willAppQuit) { // when [Ctrl|Cmd]+Q
	saveWindowState(boundsInfoPath, mainWindow);
	} else { // Minimize or hide the window for close button.
	event.preventDefault();

Confirmed that Ctrl+Q does close the application.

However these:

Click the close button.
Press Ctrl+W.
Click option in menu Window > Close.
Make my WM send a signal through the WM_DELETE protocol.

Still do not work.

[Willyfrog]

we already have an issue to track this, so I'm adding yours to the list of related issues.

[jankaifer]

Cool, sorry that I missed those.
Closing this one as duplicate of:
#1415
#1266
#90
#962