X Tutup
Skip to content

Add confirmation dialog to filesystem dock when moving or copying files#109017

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
nubels:file-system-confirmation-dialog
Nov 5, 2025
Merged

Add confirmation dialog to filesystem dock when moving or copying files#109017
Repiteo merged 1 commit intogodotengine:masterfrom
nubels:file-system-confirmation-dialog

Conversation

@nubels
Copy link
Contributor

@nubels nubels commented Jul 27, 2025

Closes godotengine/godot-proposals#7862.

This adds a confirmation dialog to the file system when the user moves or copies one or more files.
As discussed in the PR, the option to disable the confirmation was not added, and the dialog shows where the files will be moved or copied to.

CleanShot 2025-07-27 at 10 37 29

@nubels nubels requested a review from a team July 27, 2025 08:39
@AThousandShips AThousandShips added this to the 4.x milestone Jul 28, 2025
@KoBeWi
Copy link
Member

KoBeWi commented Aug 1, 2025

The dialog always says "Move selected files?", even when dragging directories.

Also the dialog looks rather barebones:
image
At least the path could be put in quotes or something, and the trailing slash should be removed.

@nubels
Copy link
Contributor Author

nubels commented Aug 5, 2025

@KoBeWi Thanks for your feedback! I've adjusted the message and removed the meta data.

Copy link
Member

@KoBeWi KoBeWi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor details left, but functionality-wise looks good.

@KoBeWi KoBeWi modified the milestones: 4.x, 4.6 Aug 5, 2025
@nubels nubels requested review from a team as code owners August 6, 2025 09:01
@AThousandShips AThousandShips removed request for a team August 6, 2025 09:10
@nubels nubels force-pushed the file-system-confirmation-dialog branch from 631357d to 3d764e5 Compare August 6, 2025 09:38
@nubels
Copy link
Contributor Author

nubels commented Aug 6, 2025

Sorry for that, this is my first time. I hope it's fixed now.

@nubels nubels requested a review from AThousandShips August 6, 2025 11:04
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally, it works as expected. Code looks good to me.

@Repiteo Repiteo merged commit f99eeb9 into godotengine:master Nov 5, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Nov 5, 2025

Thanks! Congratulations on your first merged contribution! 🎉

@sebbasim
Copy link

I understand the good intentions of this issue but this is a real pain for me. I would strongly suggest adding at least the option to turn it off in the editor settings. I am rolling back to 4.5 for now.

@Kiokurashi
Copy link

This is also a great annoyance for me. When it's large files having the confirmation makes sense, but for people like me who are constantly moving only one or two things at a time just because I didn't save it in exactly the right folder it gets to be extremely annoying. Even more so when I need to do a restructuring of my files. I don't need a confirmation for the action I'm currently doing, and all this does is make me want to handle it out of engine. Any errors caused by the latter method would be preferable to a pop up every single action.

@nubels
Copy link
Contributor Author

nubels commented Feb 2, 2026

Thanks for the feedback, I will open a PR adding a checkbox soon.

@nubels nubels deleted the file-system-confirmation-dialog branch February 24, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a confirmation dialog for moving folders in the FileSystem dock

7 participants

X Tutup