Summary: | Autosave prevents saving (without any message for the user) | ||
---|---|---|---|
Product: | [Applications] krita | Reporter: | Tiar <tamtamy.tymona> |
Component: | Usability | Assignee: | Dmitry Kazakov <dimula73> |
Status: | ASSIGNED --- | ||
Severity: | normal | CC: | darkedparts, dimula73, griffinvalley, halla, mangatengu, tgdev001 |
Priority: | NOR | ||
Version: | nightly build (please specify the git hash!) | ||
Target Milestone: | --- | ||
Platform: | Microsoft Windows | ||
OS: | Microsoft Windows | ||
Latest Commit: | https://invent.kde.org/graphics/krita/commit/45c28ddef8e09e19ce186fbfcd3ef7580ecf9ab4 | Version Fixed In: |
Description
Tiar
2019-07-20 09:29:35 UTC
Hey tiar, is this still relevant? We did change a lot about saving last summer... Dear Bug Submitter, This bug has been in NEEDSINFO status with no change for at least 15 days. Please provide the requested information as soon as possible and set the bug status as REPORTED. Due to regular bug tracker maintenance, if the bug is still in NEEDSINFO status with no change in 30 days the bug will be closed as RESOLVED > WORKSFORME due to lack of needed information. For more information about our bug triaging procedures please read the wiki located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging If you have already provided the requested information, please mark the bug as REPORTED so that the KDE team knows that the bug is ready to be confirmed. Thank you for helping us make KDE software even better for everyone! This bug has been in NEEDSINFO status with no change for at least 30 days. The bug is now closed as RESOLVED > WORKSFORME due to lack of needed information. For more information about our bug triaging procedures please read the wiki located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging Thank you for helping us make KDE software even better for everyone! Still the case (and I don't think any saving changes last year were relevant; they were mostly changes in plugins for specific formats, not the general saving strategy). *** Bug 425584 has been marked as a duplicate of this bug. *** The duplicate bug is most probably caused by the same thing. It mentions that Export during Save doesn't work, either. Both are quite dangerous for the user, I believe. *** Bug 435204 has been marked as a duplicate of this bug. *** Is this still the case? Yes, More generally (see my duplicate https://bugs.kde.org/show_bug.cgi?id=425584) you can't queue saving operations. For example you can't save and export in parallel. Only the first command will be taken. I sometimes lose quite some work because I think a save was done but was not. Dear Bug Submitter, This bug has been in NEEDSINFO status with no change for at least 15 days. Please provide the requested information as soon as possible and set the bug status as REPORTED. Due to regular bug tracker maintenance, if the bug is still in NEEDSINFO status with no change in 30 days the bug will be closed as RESOLVED > WORKSFORME due to lack of needed information. For more information about our bug triaging procedures please read the wiki located here: https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging If you have already provided the requested information, please mark the bug as REPORTED so that the KDE team knows that the bug is ready to be confirmed. Thank you for helping us make KDE software even better for everyone! This leads to sometimes uncool data loss. It's easy to reproduce so if you need me to try anything... I've tried to reproduce this, but... I couldn't. This is what I saw happening: I set autosave to 1 minute, I saw the autosave message, pressed ctrl-s when that was visible and the autosave's save progress bar was in progress, I pressed ctrl-s, and my save started immediately after the autosave. Here is a way to reproduce easily as this is not directly related to autosave but concurrent saving commands: -Prepare a big canvas (big enough for the saving as a png to take several seconds). -export it to png -CTRL + s before the finished saving popup appears The file is not saved (still the star in the title etc.) Okay, I've managed to reproduce this now. Btw, note that if you regularly want to both export and save the .kra, you can just enable the checkbox "Also save as kra" in the png export dialog. Definitely will give it a try as I often do need that. However concurrent saving usecases are so numerous I'm glad you could reproduce. Merge request: https://invent.kde.org/graphics/krita/-/merge_requests/1410 A possibly relevant merge request was started @ https://invent.kde.org/graphics/krita/-/merge_requests/1410 Git commit 45c28ddef8e09e19ce186fbfcd3ef7580ecf9ab4 by Dmitry Kazakov, on behalf of Halla Rempt. Committed on 31/05/2022 at 12:10. Pushed by dkazakov into branch 'master'. Warn the user if a save operation is aborted It's possible to try to save during autosave or try to export when a save is going on, and in that case the savingMutex is locked and we'd cancel() the saving operation, silently, as if the user had pressed cancel in the export dialog. Now we can return not just a bool, but succes, failure or busy from initiateSaving depending on whether the the document was busy, saving succeeded or failed. M +26 -23 libs/ui/KisDocument.cpp M +2 -2 libs/ui/KisDocument.h M +2 -1 libs/ui/KisImportExportErrorCode.cpp M +3 -0 libs/ui/KisImportExportErrorCode.h M +6 -0 libs/ui/KisImportExportUtils.h M +4 -3 libs/ui/KisMainWindow.cpp https://invent.kde.org/graphics/krita/commit/45c28ddef8e09e19ce186fbfcd3ef7580ecf9ab4 I'm reopening this bug, because we agreed that we need a proper save operation queueing/compression. The fix in the commit avoids the data loss, but makes UIX not very user friedly. I'm assiging it to you :P |