-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Ensure filename is possibly centered below file icons in grid view #12173
Conversation
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.
Tested and works 👍
I agree with @juliushaertl . |
Look at the grid view in the file picker: Furthermore we can’t view a file in isolation as in the example @juliushaertl. Check this out:
How so? The name is literally closer to most of the image when it’s to the right, and as close to the related actions. @skjnldsv in your screenshot, the folder and the middle one of the images miss their Share icon – that’s not right? Also – all of your screenshots show thumbnails. ;) What about all the different filetype icons, of which none fill the whole preview area or are square? |
I'm fine with the filepicker like that, it looks ok. ;)
Share can be disabled :)
Exactly, most of the files should have thumbnails or so.
|
A "no" from me. Centering on the filepicker looks great because everything is centered. Having icons aligned to one edge and the title not aligned to any edge doesn't make sense to me. Also, in all those screenshots - icons are way to far from each other! Why is that? Why not give them their normal area (44px width) and place them right next to each other. Icons are buttons - hence they need to be close enough to minimize mouse movement yet far enough to give good accessibility. Predefined size makes sure the latter is achieved. |
They are 44px appart :) |
Aligning the icons to the left? They should absolutely be centered in the clickable area. Anyhow, I’ll close this then. |
No, making them bigger so they match the same size as any thumbnail :) |
Actually reopening because the current state looks even worse on the link sharing page: Here’s another proposal of centering, which looks much more balanced, no? (Manual mockup as it needs to be centered, take as much space as possible, and need to take into account the width of the 3-dot menu): @pixelipo @skjnldsv @juliushaertl what are your alternative suggestions – with mockups please? Or do you think the current state is the best?
@skjnldsv This is exactly the opposite of what looks good though, and we discussed at 'In grid view, file type icons are too big compared to image previews' #11902. Feel free to try out making the folder icons and the other filetype icons even bigger – it looks very unbalanced. |
I would leave it left aligned. Having it centered looks weird to me, since it cannot always be centered due to the share/action icons and then the name ends up being left aligned anyway. This then just looks like no alignment at all (short names are really centered, very long ones are left aligned and medium length names are neither of both).
This is a matter of perspective. The action icons do not belong to the name, but to the folder/file in general. So the name and the action icons need to be closer to their own folder icon/thumbnail than to the others, but this is the case already.
|
I still maintain my position. The tiny shift on the left is not as dramatic as it sound.
Yup I agree :) EDIT: one thing we could do is lower the size of non-generated previews. This looks further away of the filename and lower the feeling of misalignment. |
@skjnldsv so when we look at the file picker, it looks great centered below, right? Everyone who comments and suggests stuff: Please make sure to look at all the places the view is used:
We should find something that works nicely and consistently. |
My point here is that even if you center the name in the Files app, it is not really always centered, because long names have to move to the left to make space for the share icon. So it is not even consistent within the same view anymore. I think this is worse than having an inconsistency between different views. |
Moved to 16. If there is a bugfix then we can backport this. |
As discussed in #12135 and in the design meeting at the Contributor Week we will simplify the Grid view, which will ensure that the focus is on the filetype icon/preview and the filename. |
So what is the status here? |
Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
e489e21
to
e5c1049
Compare
@jancborchardt I had a look at this and moved the filename to a proper centering related to the icon. The share indicators are also removed now as discussed in #12135 Looks much cleaner and properly aligned to me now. So 👍 from my side. |
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.
Fine by me now 😉
Very nice! This is a nice solution! |
This was already intended in the pull request, not sure where this got lost. We do use the text-align: right; for the text already, but the element of the text didn’t take full width so stayed on the left. 😉
This is also to make it in line with what we do in the file picker #12154
Currently left-aligned, looks strange and the names float away from the preview/icon and actions:
Now really right-aligned, sticking to the actions and the preview:
Please review @nextcloud/designers