All the code logic that has been written under the onClosing: scope is not even considered, if we use Kirigami.ApplicationWindow . Using Kirigami.AbstractApplicationWindow instead to fix the same.
Looks fine to me. Maybe Judit can confirm this patch?
Looks fine to me, great to see it being improved!
Does this look good enough to be accepted?
Pushed to master: https://commits.kde.org/marble/d60718c0864f7686701ba45585a52ac45b2f79c0