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

folder to small #60

Closed
theScrabi opened this issue Mar 21, 2018 · 4 comments
Closed

folder to small #60

theScrabi opened this issue Mar 21, 2018 · 4 comments
Labels
Category:Enhancement Add new functionality

Comments

@theScrabi
Copy link

theScrabi commented Mar 21, 2018

Hola chicos,

I learned about this design from the blog. I find it nice, but I instantly saw some things I'd like to be better. I found that the file icons are just to small compared to the rest of the UI elements. Example:
37416039-20817b4c-27ad-11e8-9f14-cbe12936fd64

You can see that the file icons are even a bit smaller than the checkbox beside it. I don't like it, because the meaning of the icon is harder to detect, and ... it just does not look right.

Actually the size of the icons as they are right now is pretty good I think:

bildschirmfoto vom 2018-03-21 21-52-22

@felixheidecke felixheidecke added the Category:Enhancement Add new functionality label Apr 19, 2018
@DeepDiver1975
Copy link
Member

We need a general UX overhault on the file manager.

What I this like these days in the old files app is the change in behavior depending on the file type.
And also depending on where the user clicks in the file row (checkbox, icon, name, row itself)

  1. I suggest to first of all to disallow apps to change the default action.
  2. Default action on files shall open the sidebar. Any further actions are to be taken there
  3. Default action on folders is to go inside. Details page to be open via the [...] menu
  4. I'm uncertain about the multiple file actions idea. How often is it used? Can we limit the actions? Only delete selected? Background: implementation is time consuming and what is the benefit for it?

@DeepDiver1975
Copy link
Member

Once the full row is connected to one action only the height of the row is relevant and not the icon size.

@DeepDiver1975
Copy link
Member

refs: #149 (comment)

@DeepDiver1975
Copy link
Member

fixed from my pov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category:Enhancement Add new functionality
Projects
None yet
Development

No branches or pull requests

3 participants