SUMMARY When closing a minimized document in subwindow mode, no save dialog is shown, causing loss of any changes made to a document. Either by pressing close button in subwindow frame or by menu/keyboard shortcut. On my machine the bug is present in krita-5.1.5-x86_64.appimage, krita-5.2.0-beta2-ebc38827e2-x86_64.appimage, krita-5.3.0-prealpha-f46c04aa3e-x86_64.appimage and also on macOS with Krita 5.1.5 and 5.2 beta2. STEPS TO REPRODUCE 1. Activate subwindow mode in settings 2. Create a new document and draw something 3. If on macOS: Create another new document and draw something 4. Minimize document(s) and close them. OBSERVED RESULT Documents close and no save dialog is displayed EXPECTED RESULT Save dialog SOFTWARE/OS VERSIONS macOS: 10.13.6 Linux/KDE Plasma: openSUSE Tumbleweed 20230828 KDE Plasma Version: 5.27.7 KDE Frameworks Version: 5.109.0 Qt Version: 5.15.10
I can confirm this bug on Windows. And since it is a data-loss, I will also escalate it to a release_blocker status.
This is a regression caused by a318e3f9491f773d09c452b1b3681f71d3152009, which was a bugfix for bug 392930
Git commit 95a2e1a8b70410e0381274d13cd78ea7df486acc by Dmitry Kazakov. Committed on 11/09/2023 at 16:07. Pushed by dkazakov into branch 'krita/5.2'. Don't allow closing hidden document views without confirmation The original fix was supposed to ease the multimonitor layouts somehow, but it seems like the feature is just broken and this check does not really change much. Related: bug 392930 Original bug whose fix caused the issue: M +2 -2 libs/ui/KisView.cpp https://invent.kde.org/graphics/krita/-/commit/95a2e1a8b70410e0381274d13cd78ea7df486acc
Git commit 74fd46e4238390c20f703a16ea5dde826bfbc747 by Dmitry Kazakov. Committed on 11/09/2023 at 16:07. Pushed by dkazakov into branch 'krita/5.2'. Don't hide already hidden main windows That causes unbalanced show/hide events to be delivered, which is not particularily a bug, but makes our life much more complicated. M +3 -1 libs/ui/KisWindowLayoutResource.cpp https://invent.kde.org/graphics/krita/-/commit/74fd46e4238390c20f703a16ea5dde826bfbc747
Git commit 1c0f18bc706889a9e02183e462fd37cf4f7e1d3c by Dmitry Kazakov. Committed on 11/09/2023 at 16:16. Pushed by dkazakov into branch 'master'. Don't allow closing hidden document views without confirmation The original fix was supposed to ease the multimonitor layouts somehow, but it seems like the feature is just broken and this check does not really change much. Related: bug 392930 Original bug whose fix caused the issue: M +2 -2 libs/ui/KisView.cpp https://invent.kde.org/graphics/krita/-/commit/1c0f18bc706889a9e02183e462fd37cf4f7e1d3c
Git commit 15f4024296fd720b5c03b7a6f9676bfd71bcf6c0 by Dmitry Kazakov. Committed on 11/09/2023 at 16:16. Pushed by dkazakov into branch 'master'. Don't hide already hidden main windows That causes unbalanced show/hide events to be delivered, which is not particularily a bug, but makes our life much more complicated. M +3 -1 libs/ui/KisWindowLayoutResource.cpp https://invent.kde.org/graphics/krita/-/commit/15f4024296fd720b5c03b7a6f9676bfd71bcf6c0