Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FileSystemDock: Don't update current path on rename when file list has focus #81007

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

coumcashier
Copy link
Contributor

@coumcashier coumcashier commented Aug 25, 2023

fix #80940

FileSystemDock: Don't update current path and path edit when the file list has focus. So that when renaming from the file list, we don't cd into the renamed folder automatically.

@coumcashier
Copy link
Contributor Author

demo.mp4

@coumcashier
Copy link
Contributor Author

It might be good if we don't lose currently highlighted folder in the tree view after rename.

@fire fire changed the title rename: don't update current path when file list has focus Rename: don't update current path when file list has focus Aug 26, 2023
@akien-mga akien-mga changed the title Rename: don't update current path when file list has focus FileSystemDock: Don't update current path on rename when file list has focus Aug 26, 2023
@KoBeWi
Copy link
Member

KoBeWi commented Oct 1, 2023

Needs rebase.

@coumcashier
Copy link
Contributor Author

tested on master issue still exist.
rebased.

FileSystemDock: Don't update current path and path edit when the file list has focus.
So that when renaming from the file list, we don't cd into the renamed folder automatically.
@KoBeWi

This comment was marked as resolved.

@coumcashier coumcashier closed this Oct 2, 2023
@coumcashier coumcashier reopened this Oct 2, 2023
@coumcashier coumcashier marked this pull request as draft October 2, 2023 10:25
@coumcashier coumcashier marked this pull request as ready for review October 2, 2023 10:29
@coumcashier
Copy link
Contributor Author

Its back now.
lessons learnt.

@akien-mga akien-mga merged commit 0d4fe74 into godotengine:master Oct 2, 2023
30 checks passed
@akien-mga
Copy link
Member

Thanks! And congrats for your first merged Godot contribution 🎉

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.

renaming a folder and pressing enter enters the folder
3 participants