onlyquitonempty: refactoring patch to only take client windows into
consideration when deciding whether or not to allow dwm to quit As per the original patch https://dwm.suckless.org/patches/onlyquitonempty/ it used XQueryTree to get a count of the number of windows open to determine whether to allow the window manager to exit. This meant that the empty quit count variable would have to take into account background windows such as the bar, which has side effects like plugging in another monitor could mean that you would not longer be allowed to quit dwm until the monitor is removed. Likewise a systray and each systray icon would give a +1 to the number of windows in the system. This is unintuitive to understand and convoluted to explain, hence the refactoring here to use the more sane approach of only counting the number of client windows that the window manager manages. This is an old idea which was intentionally not added to dwm-flexipatch due to the aim of staying true to the original patch (as in if you were to patch that manually you would get the same experience as you had when trying the patch out in dwm-flexipatch). This is ref. discussion in #194.pull/207/head
parent
ec6a64a64f
commit
67fc80803d
Loading…
Reference in New Issue