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

Ensure filename is possibly centered below file icons in grid view #12173

Merged
merged 2 commits into from
Jan 15, 2019

Conversation

jancborchardt
Copy link
Member

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:
file names current left aligned

Now really right-aligned, sticking to the actions and the preview:
file names centered in grid view

Please review @nextcloud/designers

Copy link
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

@juliusknorr
Copy link
Member

This looks a bit odd to me. I think we should either center it in the real middle of the preview (which won't work because of the actions) or keep it left aligned. There should be a visual connection between the preview and the name. With the semi-center alignment, this connection is lost.

image

@skjnldsv
Copy link
Member

skjnldsv commented Nov 1, 2018

I agree with @juliushaertl .
Centering on an unknown width is worse. It is too inconsistent :)
I would rather make sure the folder icons are aligned with the text like the thumbnails.

capture d ecran_2018-11-01_12-47-39

@jancborchardt
Copy link
Member Author

jancborchardt commented Nov 4, 2018

Look at the grid view in the file picker:
screenshot from 2018-11-04 01-08-41
The text is center-aligned below the icon because the icons themselves are centered too.

Furthermore we can’t view a file in isolation as in the example @juliushaertl. Check this out:
screenshot from 2018-11-04 01-12-15
Also in my screenshots in the original post: Some file/folder names like "Notes" or "Bahn" are visually much closer to the action icons of other elements than their own. This breaks the Gestalt Law of Proximity.

With the semi-center alignment, this connection is lost.

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?

@skjnldsv
Copy link
Member

skjnldsv commented Nov 4, 2018

Look at the grid view in the file picker:

I'm fine with the filepicker like that, it looks ok. ;)

@skjnldsv in your screenshot, the folder and the middle one of the images miss their Share icon – that’s not right?

Share can be disabled :)

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?

Exactly, most of the files should have thumbnails or so.
I will suggest we tune the filetypes icons, which should be the only one not filling up the whole square, to match the alignement on the left 👍

I would rather make sure the folder icons are aligned with the text like the thumbnails.

@MorrisJobke MorrisJobke mentioned this pull request Nov 6, 2018
29 tasks
@pixelipo
Copy link
Contributor

pixelipo commented Nov 6, 2018

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.

@skjnldsv
Copy link
Member

skjnldsv commented Nov 6, 2018

Also, in all those screenshots - icons are way to far from each other! Why is that?

They are 44px appart :)
If we make them closer, we need to space them, or use margin hack, but then they will have a margin conflict between them :(

@jancborchardt
Copy link
Member Author

I will suggest we tune the filetypes icons, which should be the only one not filling up the whole square, to match the alignement on the left +1

Aligning the icons to the left? They should absolutely be centered in the clickable area.

Anyhow, I’ll close this then.

@jancborchardt jancborchardt deleted the filename-center branch November 7, 2018 11:05
@skjnldsv
Copy link
Member

skjnldsv commented Nov 7, 2018

Aligning the icons to the left? They should absolutely be centered in the clickable area.

No, making them bigger so they match the same size as any thumbnail :)

@MorrisJobke MorrisJobke removed this from the Nextcloud 15 milestone Nov 7, 2018
@jancborchardt
Copy link
Member Author

jancborchardt commented Nov 7, 2018

Actually reopening because the current state looks even worse on the link sharing page:
sharing folder 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):
share link center

@pixelipo @skjnldsv @juliushaertl what are your alternative suggestions – with mockups please? Or do you think the current state is the best?

No, making them bigger so they match the same size as any thumbnail :)

@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.

@jancborchardt jancborchardt restored the filename-center branch November 7, 2018 12:11
@jancborchardt jancborchardt reopened this Nov 7, 2018
@jancborchardt jancborchardt added this to the Nextcloud 15 milestone Nov 7, 2018
@raimund-schluessler
Copy link
Member

raimund-schluessler commented Nov 7, 2018

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).

Also in my screenshots in the original post: Some file/folder names like "Notes" or "Bahn" are visually much closer to the action icons of other elements than their own. This breaks the Gestalt Law of Proximity.

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.

Maybe it would look less weird to you, if the folder icon would not have so much white-space around but would look similar in size to a file preview image. I saw that was neglected already, maybe moving the file type icon a bit down would do better then?

@skjnldsv
Copy link
Member

skjnldsv commented Nov 7, 2018

I still maintain my position. The tiny shift on the left is not as dramatic as it sound.
It's still far better than the center I think.
I would leave it like that.

No, making them bigger so they match the same size as any thumbnail :)

@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.

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.
capture d ecran_2018-11-07_14-22-37

@jancborchardt
Copy link
Member Author

@skjnldsv so when we look at the file picker, it looks great centered below, right?
Now just adding one action icon (in share link) results in the text needing to go to the left? That doesn’t seem quite right and not consistent.

Everyone who comments and suggests stuff: Please make sure to look at all the places the view is used:

  • Files app (with share icon and 3-dot icon): currently left-aligned
  • Shared folder (with only 3-dot icon): currently left-aligned
  • File picker (no icons): currently centered

We should find something that works nicely and consistently.

@raimund-schluessler
Copy link
Member

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.

@MorrisJobke MorrisJobke mentioned this pull request Nov 8, 2018
24 tasks
@MorrisJobke
Copy link
Member

Moved to 16. If there is a bugfix then we can backport this.

@jancborchardt
Copy link
Member Author

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.

@rullzer
Copy link
Member

rullzer commented Jan 14, 2019

So what is the status here?

jancborchardt and others added 2 commits January 14, 2019 20:19
Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr
Copy link
Member

@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

image

Looks much cleaner and properly aligned to me now. So 👍 from my side.

Copy link
Member

@juliusknorr juliusknorr left a 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 😉

@skjnldsv
Copy link
Member

Very nice! This is a nice solution!
Thanks both of you!

@MorrisJobke MorrisJobke merged commit 6ac66ea into master Jan 15, 2019
@MorrisJobke MorrisJobke deleted the filename-center branch January 15, 2019 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug design Design, UI, UX, etc. feature: files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants