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

Speed up filesystem dock operations #87917

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KoBeWi
Copy link
Member

@KoBeWi KoBeWi commented Feb 3, 2024

Fixes #50576

godot.windows.editor.dev.x86_64_JhN90GK3xC.mp4

Moving, duplicating, renaming and creating folders is now instant.

In fact it's just a trick - the Tree is updated instantly upon action, but the rescan still runs in the background. When it finishes, it causes Tree sync with the actual filesystem. There are still some bugs like files appearing in wrong place or wrong selections, so putting as draft.

@KoBeWi
Copy link
Member Author

KoBeWi commented Feb 23, 2024

Finished 😬
Making sure every TreeItem updates when needed required unspeakable amount of ugly code. But now item positions and selections are consistent after action and after the background rescan.

I didn't touch ItemList view, not sure if there is an easy way to handle it. Also if you do file operations on modified TreeItems before rescan finished they will take no effect. I don't feel like improving it anymore though.

@KoBeWi KoBeWi marked this pull request as ready for review February 23, 2024 23:29
@KoBeWi KoBeWi requested review from a team as code owners February 23, 2024 23:29
@KoBeWi KoBeWi modified the milestones: 4.3, 4.x Feb 23, 2024
@adamscott
Copy link
Member

@AThousandShips Did you take a look at this PR?

@AThousandShips
Copy link
Member

I did glance at it but will take a fresh look

@AThousandShips AThousandShips self-requested a review March 8, 2024 18:44
@adamscott adamscott self-requested a review March 8, 2024 18:50
@adamscott

This comment was marked as resolved.

@akien-mga akien-mga modified the milestones: 4.x, 4.4 May 28, 2024
@KoBeWi KoBeWi marked this pull request as draft June 25, 2024 16:15
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.

Folder operation in FileSystem dock cause all assets to be rescanned 3.3.2
5 participants