Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

number of items as last entry in the list #993

Closed
jancborchardt opened this Issue · 18 comments

6 participants

@jancborchardt
Collaborator

An idea from iOS apps, probably also used elsewhere: The last entry in the list always says how many items there are in the list, for example »34 entries«.

It might make sense to have this as the last entry in Files as well, I’m just not sure if it should be the complete number of files also including those of subfolders, or just »5 folders + 32 files« stating what is in the current level.

This would also be cool for other apps like Bookmarks @blizzz, Contacts @tanghus, Mail @DeepDiver1975, for the Users settings (number of users) and probably more.
And also in the mobile apps cc @davivel @rperezb

@davidak
Collaborator

Like described in my Issue #2455 my idea was to show the Number of Files in the Top Bar where "Neu" and Upload is.

Here is a Mockup of both:
Anzahl_Dateien_Mockup_davidak

I think in the top Bar you get the relevant Information quicker and it don't need more space linke under the items.
This Space is now empty.

To show the number of items inside a folder it can be displayed in the bubble:
OwnCloud_Anzahl_Dateien_Ordner

What do you Think?

@jancborchardt
Collaborator

@davidak yeah, nice mockups! It’s almost what I mean.

For the first one, the »1 folder & 4 files« shouldn’t be displayed twice. Only having it at the bottom is enough. Then also the language should be simplified and combined with the size, for example. »1 folder & 4 files, 800 MB«. It should also be grey or have opacity:.5; to be different from the rest of the files, and probably also centered instead of left aligned. Also there doesn’t need to be a bottom border for that entry.

For the second mockup, displaying the »95 files« in the bubble, great one! I didn’t even think about this.

@davidak what do you think about that? And do you want to implement it? Would be really cool!

@jancborchardt
Collaborator

Also, maybe it’s good to have this in a general way, so it can also be used by other apps which have listings where it’s good to have the number on the bottom. As mentioned above maybe for Mail, Contacts, User management, Bookmarks and so on.

@blizzz
Collaborator

isn't it better to have it always on the top? Avoids unnecessary scrolling in folders with a lot of content

@davidak
Collaborator

@jancborchardt it is displayed twice to show both ideas: yours and mine!

If it is at the bottom, there could be a ∑ (summe) icon.

unfortunately i am not so experienced in php-programming that i could implement that.

@jancborchardt
Collaborator

@blizzz the information is not that important. Currently it’s not shown at all. Showing it at the bottom suffices for the first implementation, and we can see from there.

@davidak no one understands the summe icon. ;) Just »4 folders & 8 files, 340 MB« is enough.
If you want to give it a try and implement it I’m sure we can help you if you have problems. We have the IRC channel #owncloud-dev and the mailing list. You have to start somewhere. ;)

@MorrisJobke MorrisJobke closed this in #3915
@davidak
Collaborator

@Kondou-ger thanks for your work. I am pleased to see it in action. could be post a screenshot?

@Kondou-ger
Collaborator

@davidak real pros check it out themselves ;)
screenshot380

@jancborchardt
Collaborator

@Kondou-ger can you make the entry have a bit more padding to the top and bottom? Otherwise it looks like it’s a list entry. Also, it should not have a bottom border. Otherwise awesome! :)

@jancborchardt jancborchardt reopened this
@davidak
Collaborator

@Kondou-ger looks great. maybe the & can be text like 'and' and what @jancborchardt said is also good.

im not that edgy that i use the master branch.

@Kondou-ger Kondou-ger referenced this issue from a commit
@Kondou-ger Kondou-ger Improve filelist summary
 - Remove bottom border of summary

 - Increase padding of summary

 - Use text instead of '&'

Fix #993
06b13a1
@jancborchardt
Collaborator

(Closing this here in favor of #3926. :)

@Kondou-ger Kondou-ger referenced this issue from a commit
@Kondou-ger Kondou-ger Improve filelist summary
 - Remove bottom border of summary

 - Increase padding of summary

 - Use text instead of '&'

 - No hover effect for the filelist-summary

 - Improve l10n aspects

 - Apply padding to top and bottom

 - Use background-color: transparent;

Fix #993
67f8486
@Kondou-ger Kondou-ger referenced this issue from a commit
@Kondou-ger Kondou-ger Improve filelist summary
 - Remove bottom border of summary

 - Increase padding of summary

 - Use text instead of '&'

 - No hover effect for the filelist-summary

 - Improve l10n aspects

 - Apply padding to top and bottom

 - Use background-color: transparent;

Fix #993
4aabf51
@Kondou-ger
Collaborator

Since #4118 removed the tipsy for filesizes, where should the directory count go now?

@MorrisJobke
Collaborator

@Kondou-ger Can you open a new issue, otherwise this will get lost.

@Kondou-ger
Collaborator

Why open a new issue …
We don't just randomly close and reopen other issues as well. Just that there was some activity in here doesn't justify this …

@Kondou-ger Kondou-ger reopened this
@Kondou-ger
Collaborator

Reopened this, because
img
isn't implemented yet/hasn't been done by my PRs.

@jancborchardt
Collaborator

@Kondou-ger hm, didn’t think about that. Maybe good call to add the tipsy for file sizes again, but only for folders? And then it would only show the »95 files«. That is, if we want to show the number of files at all – we didn’t have it before either. But I guess it’s good info?

What do you think? If you think it’s good to add it back then it would be cool if you can do that. :) cc @owncloud/designers

@DeepDiver1975

Besides all the fancy-ness: retrieving the number of files/folders contained is not cheep either.

In terms of simplicity and performance I'd just skip this.

@jancborchardt
Collaborator

Right, forgot to mention that – if we don’t already have that number or cache it somewhere, I’d rather we not do this, for performance reasons.

@Kondou-ger Kondou-ger referenced this issue from a commit
@Kondou-ger Kondou-ger Improve filelist summary
 - Remove bottom border of summary

 - Increase padding of summary

 - Use text instead of '&'

 - No hover effect for the filelist-summary

 - Improve l10n aspects

 - Apply padding to top and bottom

 - Use background-color: transparent;

Fix #993
be87044
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.