-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Improve folder enumeration speed #4922
Conversation
Hide progress bar after enumeration
@gave92 The new details layout will replace the old one in v2. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like a good improvement as long as we remove the old details layout before v2
@gave92 It seems like the status bar isn't updating anymore. Another issue is that everything seems very choppy when loading large directories. |
@yaichenbaum is status bar working in |
It looks like it's an issue in |
@yaichenbaum is this still an issue? If so could you clarify? I would not expect it to be "more choppy" than the current. |
It appears to be working better now, I noticed a different issue where the app seems to scroll down after loading the system 32 directory in the details layout. |
Could you get me a screen recording of this? I have trouble reproducing |
I reset Files and I can't reproduce it anymore 😂 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Resolved / Related Issues
For #4180
Details of Changes
Warning
These changes creates issues with the "current" details view, I find the app crashing every 5 folders.
This only happens with the WCT datagrid layout mode, the "new" details view works good.
This actually looks like a bug with WCT and has been known for a while (#4304) but these changes seem to make the crash more common.
Depending on how we plan to ship the next version (new or old details view) it may make sense to wait before doing this change, but I'd love to get some feedbacks on how well this PR work for you.
Validation
How did you test these changes?