-
Notifications
You must be signed in to change notification settings - Fork 489
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
Files page: buttons and inconsistencies #1122
Comments
@lidel then, if you went to pins section, how'd you reach 'Files'? You could certainly click on the navigation bar on the left, but... |
Ah, yes, i thought i was forgetting some conversations. I think @lidel's intuition sounds good: perhaps we make the Ignore the following if it represents an utter lack of proper grokking of things! … |
Calling scope to the dropdown and path to everything that follows it, I don't believe it is a good idea. On my mind, changing the scope would keep the path. I don't know if what I just said is understandable. Let me know. Also, there are situations (when the user writes a |
yes, but not for pins: they do not share file paths with MFS, only low level CIDs are used. pull-down would be a good UI for switching between "files" and "blocks" views of the same path, but as @hacdias noted (if I got it right) the file path breadcrumbs on the right would have to be dropped when switching to "pins". This would produce inconsistent user experience (path getting lost when switching between views). We can revisit pull-down when adding "blocks" view/stats, but for the problem at hand I would start with a small change and replace home icon with size of MFS, and group+indicate clickable items for easy switching between "files" and "pins" (as suggested by @ericronne in past): MFS size is not the whole picture, so I also added global "repo" disk use (files + pins + cached dags, could be also named "cached") as a read-only stat next to block count. Size values in bytes can be read via: $ ipfs files stat / | grep CumulativeSize # value for "files"
$ ipfs repo stat | grep RepoSize # value for "repo" Would that be intuitive enough? |
We've fixed the inconsistencies on the "stats"/buttons bar! Closing this |
From #1114
The text was updated successfully, but these errors were encountered: