Lack of file overwrite prompt leads to data loss #87265

Closed
opened 2021-04-07 11:51:20 +02:00 by Ludvik Koutny · 22 comments
Contributor

System Information
Operating system: Windows-10-10.0.19041-SP0 64 Bits
Graphics card: GeForce GTX 1080 Ti/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 461.92

Blender Version
Broken: version: 2.92.0, branch: master (modified), commit date: 2021-02-24 16:25, hash: 02948a2cab
Worked: (newest version of Blender that worked as expected)

Short description of error
2021-04-07_11-44-58.mp4
Highlighting the file name field with red color in the save as dialog is by no stretch of imagination sufficient data loss prevention. Literally any other software out there which deals with creation and writing of user generated data files has an actual file overwrite pop up dialog exactly for this reason. It's not difficult to accidentally press save instead of open hotkey shortcut. It happens. And Blender should have user's back in this case instead of punishing them with data loss. If I did not do daily backups, I would lose 2 days of work right here. I know this is not technically a bug, but I am still reporting it as one, since data loss is very severe issue. Not having a file overwrite warning is just flat out unacceptable in this day and age.

Aside from the relatively easy to miss red highlight, the save and load dialogs are literally identical:
saveload.png
So some additional error prevention is really appropriate here. This isn't the kind of case where you can really say "You should have read better". In case of potential data loss, it's really important that the software leans on the side of helping the user out instead of punishing them.

Exact steps for others to reproduce the error

  1. Start with a new empty .blend file
  2. Press a "Save as" hotkey shortcut
  3. Select existing .blend file and press enter
    Result: If done in rapid succession, while user makes a mistake of using a wrong shortcut, no file overwrite warning is issued and user data in overwritten blend file is destroyed.
    Expected: Blender has file overwrite warning dialogs like any other modern piece of software.
**System Information** Operating system: Windows-10-10.0.19041-SP0 64 Bits Graphics card: GeForce GTX 1080 Ti/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 461.92 **Blender Version** Broken: version: 2.92.0, branch: master (modified), commit date: 2021-02-24 16:25, hash: `02948a2cab` Worked: (newest version of Blender that worked as expected) **Short description of error** [2021-04-07_11-44-58.mp4](https://archive.blender.org/developer/F9922647/2021-04-07_11-44-58.mp4) Highlighting the file name field with red color in the save as dialog is by no stretch of imagination sufficient data loss prevention. Literally any other software out there which deals with creation and writing of user generated data files has an actual file overwrite pop up dialog **exactly for this reason**. It's not difficult to accidentally press save instead of open hotkey shortcut. It happens. And Blender should have user's back in this case instead of punishing them with data loss. If I did not do daily backups, I would lose 2 days of work right here. I know this is not technically a bug, but I am still reporting it as one, since data loss is very severe issue. Not having a file overwrite warning is just flat out unacceptable in this day and age. Aside from the relatively easy to miss red highlight, the save and load dialogs are literally identical: ![saveload.png](https://archive.blender.org/developer/F9922663/saveload.png) So some additional error prevention is really appropriate here. This isn't the kind of case where you can really say "You should have read better". In case of potential data loss, it's really important that the software leans on the side of helping the user out instead of punishing them. **Exact steps for others to reproduce the error** 1. Start with a new empty .blend file 2. Press a "Save as" hotkey shortcut 3. Select existing .blend file and press enter Result: If done in rapid succession, while user makes a mistake of using a wrong shortcut, no file overwrite warning is issued and user data in overwritten blend file is destroyed. Expected: Blender has file overwrite warning dialogs like any other modern piece of software.
Author
Contributor

Added subscriber: @Rawalanche

Added subscriber: @Rawalanche
ariacarina commented 2021-04-07 13:30:52 +02:00 (Migrated from localhost:3001)

Added subscriber: @ariacarina

Added subscriber: @ariacarina
ariacarina commented 2021-04-07 13:30:52 +02:00 (Migrated from localhost:3001)

This comment was removed by @ariacarina

*This comment was removed by @ariacarina*

Added subscriber: @dfelinto

Added subscriber: @dfelinto
vanessaaustin commented 2021-04-07 14:44:21 +02:00 (Migrated from localhost:3001)

Added subscriber: @vanessaaustin

Added subscriber: @vanessaaustin
vanessaaustin commented 2021-04-07 14:44:21 +02:00 (Migrated from localhost:3001)

This comment was removed by @vanessaaustin

*This comment was removed by @vanessaaustin*

Added subscriber: @mont29

Added subscriber: @mont29

Added subscriber: @TheRedWaxPolice

Added subscriber: @TheRedWaxPolice
Member

Added subscriber: @Harley

Added subscriber: @Harley
Member

I personally don't like the idea of adding "are you sure?" confirmations for things like this, but I do find our current notification to be weak.

Showing the filename in a red box seems to be an incorrect notification in that it seems like the name itself is in error, like might be the case if you enter forbidden characters.

I'd much rather that the "Save As" button would be changed, since clicking that is the thing we want to warn against. So I'd like to see it say "Overwrite", be in our "caution" color (is a theme setting for Info), and a warning icon - ⚠ - added too.

I personally don't like the idea of adding "are you sure?" confirmations for things like this, but I do find our current notification to be weak. Showing the filename in a red box seems to be an incorrect notification in that it seems like the name itself is in error, like might be the case if you enter forbidden characters. I'd much rather that the "Save As" button would be changed, since clicking that is the thing we want to warn against. So I'd like to see it say "Overwrite", be in our "caution" color (is a theme setting for Info), and a warning icon - ⚠ - added too.

@Harley Unfortunately, when working fast, a warning dialog is the only thing that can save you from this trap... There's a reason this is still standard everywhere...

So +1 for a warning dialog...

@Harley Unfortunately, when working fast, a warning dialog is the only thing that can save you from this trap... There's a reason this is still standard everywhere... So +1 for a warning dialog...
Author
Contributor

The thing is - Blender already sets the precedent by having a pop up prompt when closing an unsaved file. It's the same type of scenario. One could easily say "Just don't press the window close button if you are not sure you want to close an unsaved file" and perhaps make the close X button red, but that's simply not a sufficient solution.

No amount of graphical sugar will remedy this. A warning icon will be equally as inefficient as the red file name highlight. If the relatively large red rectangle does not work, the small yellow triangle hardly will. The data loss is a serious matter and therefore justifies a pop up dialog, the same way closing an unsaved, modified file does.

To kind of reinforce my point, I challenge you to find any other mainstream piece of software out there which has GUI, allows user to create content, save it into files, and does not have an overwrite confirmation pop up dialog. I am pretty sure you will have very hard time finding at least one.

The thing is - Blender already sets the precedent by having a pop up prompt when closing an unsaved file. It's the same type of scenario. One could easily say "Just don't press the window close button if you are not sure you want to close an unsaved file" and perhaps make the close X button red, but that's simply not a sufficient solution. No amount of graphical sugar will remedy this. A warning icon will be equally as inefficient as the red file name highlight. If the relatively large red rectangle does not work, the small yellow triangle hardly will. The data loss is a serious matter and therefore justifies a pop up dialog, the same way closing an unsaved, modified file does. To kind of reinforce my point, I challenge you to find any other mainstream piece of software out there which has GUI, allows user to create content, save it into files, and does not have an overwrite confirmation pop up dialog. I am pretty sure you will have very hard time finding at least one.

Added subscriber: @StephenSwaney

Added subscriber: @StephenSwaney

Unfortunately, when working fast, a warning dialog is the only thing that can save you from this trap.

Even more unfortunately, people become habituated to clicking the Are You Sure? dialog as part of their normal work flow - especially when working fast.

I suggest that in addition to theAre You Sure? dialog, we add a **Are You Really Sure You Are Sure?**dialog as confirmation.

> Unfortunately, when working fast, a warning dialog is the only thing that can save you from this trap. Even more unfortunately, people become habituated to clicking the **Are You Sure?** dialog as part of their normal work flow - especially when working fast. I suggest that in addition to the**Are You Sure?** dialog, we add a **Are You Really Sure You Are Sure?**dialog as confirmation.

Changed status from 'Needs Triage' to: 'Archived'

Changed status from 'Needs Triage' to: 'Archived'

Thanks for the report, but this is absolutely not a bug, everything is working as expected.

Design discussions on the tracker are strictly restricted to the initiative of active development team members (would be related to #user_interface module, but currently such UI/UX topics are more or less frozen anyway).

Thanks for the report, but this is absolutely not a bug, everything is working as expected. Design discussions on the tracker are strictly restricted to the initiative of active development team members (would be related to #user_interface module, but currently such UI/UX topics are more or less frozen anyway).
Author
Contributor

In #87265#1142469, @StephenSwaney wrote:

Unfortunately, when working fast, a warning dialog is the only thing that can save you from this trap.

Even more unfortunately, people become habituated to clicking the Are You Sure? dialog as part of their normal work flow - especially when working fast.

I suggest that in addition to theAre You Sure? dialog, we add a **Are You Really Sure You Are Sure?**dialog as confirmation.

This is often solved such that the default active UI element is the preventive one, so that if the dialog pops up and you just press enter, the active UI element is the one that does not overwrite, not the one that does.

I'd personally just suggest that destructive file operations such as overwrite would have the overwrite confirmation button itself displayed in red color. That would be a good indication.

> In #87265#1142469, @StephenSwaney wrote: >> Unfortunately, when working fast, a warning dialog is the only thing that can save you from this trap. > > Even more unfortunately, people become habituated to clicking the **Are You Sure?** dialog as part of their normal work flow - especially when working fast. > > I suggest that in addition to the**Are You Sure?** dialog, we add a **Are You Really Sure You Are Sure?**dialog as confirmation. This is often solved such that the default active UI element is the preventive one, so that if the dialog pops up and you just press enter, the active UI element is the one that does not overwrite, not the one that does. I'd personally just suggest that destructive file operations such as overwrite would have the overwrite confirmation button itself displayed in red color. That would be a good indication.
Author
Contributor

In #87265#1142470, @mont29 wrote:
Thanks for the report, but this is absolutely not a bug, everything is working as expected.

Design discussions on the tracker are strictly restricted to the initiative of active development team members (would be related to #user_interface module, but currently such UI/UX topics are more or less frozen anyway).

So you are implying that data loss is more or less fine? And Blender should not help the users here, but punish them instead? That's it?

If this is your approach to data loss, then can you explain why this was addressed, even though it was not technically a bug: https://developer.blender.org/T61412 ? This is the exact same class of problem.

> In #87265#1142470, @mont29 wrote: > Thanks for the report, but this is absolutely not a bug, everything is working as expected. > > Design discussions on the tracker are strictly restricted to the initiative of active development team members (would be related to #user_interface module, but currently such UI/UX topics are more or less frozen anyway). So you are implying that data loss is more or less fine? And Blender should not help the users here, but punish them instead? That's it? If this is your approach to data loss, then can you explain why this was addressed, even though it was not technically a bug: https://developer.blender.org/T61412 ? This is the exact same class of problem.
Member

Added subscriber: @filedescriptor

Added subscriber: @filedescriptor
Member

@Rawalanche

So you are implying that data loss is more or less fine? And Blender should not help the users here, but punish them instead? That's it?

No, we have rules on the bug tracker. As Bastien has said, this would not classify as a bug because the code is working as intended. So the bug tracker is just not the place to have this discussion. That doesn't mean it's not welcome, it is! Just not here. A better place would be devtalk.

@Rawalanche > So you are implying that data loss is more or less fine? And Blender should not help the users here, but punish them instead? That's it? No, we have rules on the bug tracker. As Bastien has said, this would not classify as a bug because the code is working as intended. So the bug tracker is just not the place to have this discussion. That doesn't mean it's not welcome, it is! Just not here. A better place would be devtalk.
Author
Contributor

In #87265#1142716, @filedescriptor wrote:
@Rawalanche

So you are implying that data loss is more or less fine? And Blender should not help the users here, but punish them instead? That's it?

No, we have rules on the bug tracker. As Bastien has said, this would not classify as a bug because the code is working as intended. So the bug tracker is just not the place to have this discussion. That doesn't mean it's not welcome, it is! Just not here. A better place would be devtalk.

Every. Single. Time.... I post something like this on devtalk, it ends up ignored with 0 replies. It's not a new shiny sculpt brush, or a new geometry nodes node. It's just a possibility of data loss. No one cares about that.

> In #87265#1142716, @filedescriptor wrote: > @Rawalanche > >> So you are implying that data loss is more or less fine? And Blender should not help the users here, but punish them instead? That's it? > > No, we have rules on the bug tracker. As Bastien has said, this would not classify as a bug because the code is working as intended. So the bug tracker is just not the place to have this discussion. That doesn't mean it's not welcome, it is! Just not here. A better place would be devtalk. Every. Single. Time.... I post something like this on devtalk, it ends up ignored with 0 replies. It's not a new shiny sculpt brush, or a new geometry nodes node. It's just a possibility of data loss. No one cares about that.

Hey! Not sure where the best place to bump this issue would be, but I feel like this needs to be discussed.

Particularly with how accidentally double clicking a .blend file in the save as menu will almost always result in unintended data loss – in the early days of blender, there was a save overwrite confirmation and you couldn't double click to save in the save-as menu. But over the years, Blender evolved and that consideration got understandably lost somewhere along the way.

Having a confirmation in the save-as menu when trying to save a file with the same name as an already existing file in that folder seems like it would be fairly easy to implement, would prevent many cases of data loss in the future, and bring Blender in line with a majority of other software that already have this behavior.

Hey! Not sure where the best place to bump this issue would be, but I feel like this needs to be discussed. Particularly with how accidentally double clicking a .blend file in the save as menu will almost always result in unintended data loss – [in the early days of blender, there was a save overwrite confirmation and you couldn't double click to save in the save-as menu. ](https://devtalk.blender.org/t/some-of-the-bug-triage-staff-are-very-inexperienced/14718/37?u=ben_knight) But over the years, Blender evolved and that consideration got understandably lost somewhere along the way. Having a confirmation in the save-as menu when trying to save a file with the same name as an already existing file in that folder seems like it would be fairly easy to implement, would prevent many cases of data loss in the future, and bring Blender in line with a majority of other software that already have this behavior.
Sign in to join this conversation.
No Label
Interest
Alembic
Interest
Animation & Rigging
Interest
Asset Browser
Interest
Asset Browser Project Overview
Interest
Audio
Interest
Automated Testing
Interest
Blender Asset Bundle
Interest
BlendFile
Interest
Collada
Interest
Compatibility
Interest
Compositing
Interest
Core
Interest
Cycles
Interest
Dependency Graph
Interest
Development Management
Interest
EEVEE
Interest
EEVEE & Viewport
Interest
Freestyle
Interest
Geometry Nodes
Interest
Grease Pencil
Interest
ID Management
Interest
Images & Movies
Interest
Import Export
Interest
Line Art
Interest
Masking
Interest
Metal
Interest
Modeling
Interest
Modifiers
Interest
Motion Tracking
Interest
Nodes & Physics
Interest
OpenGL
Interest
Overlay
Interest
Overrides
Interest
Performance
Interest
Physics
Interest
Pipeline, Assets & IO
Interest
Platforms, Builds & Tests
Interest
Python API
Interest
Render & Cycles
Interest
Render Pipeline
Interest
Sculpt, Paint & Texture
Interest
Text Editor
Interest
Translations
Interest
Triaging
Interest
Undo
Interest
USD
Interest
User Interface
Interest
UV Editing
Interest
VFX & Video
Interest
Video Sequencer
Interest
Virtual Reality
Interest
Vulkan
Interest
Wayland
Interest
Workbench
Interest: X11
Legacy
Blender 2.8 Project
Legacy
Milestone 1: Basic, Local Asset Browser
Legacy
OpenGL Error
Meta
Good First Issue
Meta
Papercut
Meta
Retrospective
Meta
Security
Module
Animation & Rigging
Module
Core
Module
Development Management
Module
EEVEE & Viewport
Module
Grease Pencil
Module
Modeling
Module
Nodes & Physics
Module
Pipeline, Assets & IO
Module
Platforms, Builds & Tests
Module
Python API
Module
Render & Cycles
Module
Sculpt, Paint & Texture
Module
Triaging
Module
User Interface
Module
VFX & Video
Platform
FreeBSD
Platform
Linux
Platform
macOS
Platform
Windows
Priority
High
Priority
Low
Priority
Normal
Priority
Unbreak Now!
Status
Archived
Status
Confirmed
Status
Duplicate
Status
Needs Info from Developers
Status
Needs Information from User
Status
Needs Triage
Status
Resolved
Type
Bug
Type
Design
Type
Known Issue
Type
Patch
Type
Report
Type
To Do
No Milestone
No project
No Assignees
9 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#87265
No description provided.