Force quit with no more windows closes the window manager

[OfficialKris]

Describe the bug
See title

To Reproduce
Steps to reproduce the behavior:

1. Use force-quit keybind on windows until no more windows + 1
2. See window manager quit

Expected behavior
Maybe play standard error code? Or at least do nothing.

[cloudyster]

I would not like this behavior, considering closing a number of terminals by hitting force-quit once to often.

I'd prefer (and like) a new 'role' for closing the window manager.

Edit: I've just read about #5 - but still like a role better ;-)

[OfficialKris]

@cloudyster I think maybe a log-off role which could run kill $(pgrep -o $USER) with the option to change the command in the config would be good.

However, I think it might be worth looking into a different command as this has some drawbacks like dangling/zombie processes, force quit on all processes and doesn't even kill all user related processes.

From what I could find, this seems better (I did not test this):

busctl call org.freedesktop.login1 /org/freedesktop/login1 \
       org.freedesktop.login1.Manager TerminateSession s \
       $(busctl --user get-property org.freedesktop.login1 /org/freedesktop/login1/session/self org.freedesktop.login1.Session Id | cut -d'"' -f2)

It is dependent on systemd but if a user wants, they should be able to change the command.

[BobdaProgrammer]

@OfficialKris @cloudyster I will get into this bug, it isn't the expected behaviour. It should be a very simple fix, just check if the window to be force closed is the root window and don't destroy it if it is. I also agree a role would be good. Currently I am using an eww widget I made as a powermenu so I don't have to type out a command to logout but this option is something that I will definitely implement. Also a note, the current best way to quit doWM without zombie processes is kill $(pgrep -o doWM). I changed to this command with the issue previously mentioned.

[cloudyster]

@OfficialKris : Thanks, I'm not sure on which systems killall is available or not but I'm using killall doWM on RedHat Linux 8.