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

Showing folder size in details view #3944

Closed
pbakes opened this issue Mar 10, 2021 · 11 comments · Fixed by #7244
Closed

Showing folder size in details view #3944

pbakes opened this issue Mar 10, 2021 · 11 comments · Fixed by #7244

Comments

@pbakes
Copy link

pbakes commented Mar 10, 2021

There are so many third-party file explorer's that offer showing folder size, e.g. Explorer++, Xinorbis, WizTree, TreeSize, still nothing that allows you to view folder size in Explorer. Files, as far as the UI is concerned, is much better than all those alternatives. It would be nice to see that functionality implemented.

12059

@gave92
Copy link
Member

gave92 commented Mar 10, 2021

This is something we have looked at in the past and decided against implementing due to the performance issues involved.
[Internal comment] Not sure if the new caching feature could change this, perhaps doing it in the background from the fulltrust process? Perhaps an alternative way is to provide a "calculate size" button to calculate it on-demand?

@d2dyno1
Copy link
Member

d2dyno1 commented Mar 10, 2021

Perhaps a setting for this would suffice?

@R3voA3
Copy link
Contributor

R3voA3 commented Mar 11, 2021

It's important that the preview is not slowed down any further, it's already slow for folder with many child folders.

@yaira2
Copy link
Member

yaira2 commented Apr 11, 2021

@gave92 Do you think that if we added a calculate button, that it should calculate all items, or would it be an individual option for each folder?

@gave92
Copy link
Member

gave92 commented Apr 11, 2021

@yaichenbaum I was thinking about a button that calculates size for all currently selected folders.

@cinqmilleans
Copy link
Contributor

@gave92 @yaichenbaum @d2dyno1
I did a trial implementation. This does not reduce performance so much because the sizes are calculated after displaying the rest. I do think, however, that this should be a setting, (disabled by default). The implementation only calculates once per element and displays in the size column and footer.

FolderSize.mp4

@yaira2
Copy link
Member

yaira2 commented Nov 30, 2021

@cinqmilleans that doesn't look too bad, does it have any impact on performance at all?

@cinqmilleans
Copy link
Contributor

@yaichenbaum I did not see any noticeable loss. The sizes update last without blocking the user interface. We can therefore do everything even if the sizes have not finished loading: open a sub-folder, rename, ....

@yaira2
Copy link
Member

yaira2 commented Nov 30, 2021

@yaichenbaum I did not see any noticeable loss. The sizes update last without blocking the user interface. We can therefore do everything even if the sizes have not finished loading: open a sub-folder, rename, ....

@cinqmilleans do you want to add this as an experimental option?

@cinqmilleans
Copy link
Contributor

@yaichenbaum Yes of course. I prepare this.

@yaira2
Copy link
Member

yaira2 commented Dec 8, 2021

@pbakes this issue should be resolved with #7244.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants