server icon indicating copy to clipboard operation
server copied to clipboard

Make the grid view more usable by handling longer filenames better

Open artonge opened this issue 1 year ago • 1 comments

Improve the overall design of the grid view. @nextcloud/designers for mockups

  • Main issue is that file names beyond ~10 characters can't be read.
  • If accessibility can be fixed/improved easily, do that too, but only as low priority.

artonge avatar May 22 '24 11:05 artonge

Example from google drive: image

szaimen avatar May 22 '24 14:05 szaimen

@nextcloud/designers can we have some proper mockups for that?

artonge avatar May 29 '24 13:05 artonge

Example Dropbox:

Grid view Big grid view
image image

szaimen avatar May 30 '24 07:05 szaimen

Onedrive: image

szaimen avatar May 30 '24 08:05 szaimen

Penpot: image

szaimen avatar May 30 '24 08:05 szaimen

I like google's approach the best

marcoambrosini avatar May 30 '24 08:05 marcoambrosini

Currently working on some ideas in https://design.penpot.app/#/workspace/34e522a0-5cc4-11ec-826f-e949c75b760d/064214a4-70ce-8065-8004-6d675c5cd97c

szaimen avatar May 30 '24 09:05 szaimen

This is how the final mockup without previews looks: image and here with previews: image

szaimen avatar Jun 20 '24 10:06 szaimen

I'm not sure whether I should open a new issue or not for this but in Nextcloud 29 at least, when filenames are very long it is not possible to see the full name on mobile in any place AFAIK.

jmigual avatar Oct 02 '24 09:10 jmigual

@jmigual can you provide screenshots?

artonge avatar Oct 02 '24 10:10 artonge

Sure! Here they are:

Screenshot_2024-10-02-13-30-45-15_40deb401b9ffe8e1df2f1cc5ba480b12.jpg

Screenshot_2024-10-02-13-30-39-46_40deb401b9ffe8e1df2f1cc5ba480b12.jpg

The biggest problem IMO is that it's never possible to see the full name of the file. Not even on the details panel.

jmigual avatar Oct 02 '24 11:10 jmigual

@jmigual Ok, this is not related to this pull request. You can open a new issue if you like :).

artonge avatar Oct 02 '24 13:10 artonge