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

📍 File picker dialog improvements #38926

Closed
4 tasks done
jancborchardt opened this issue Jun 21, 2023 · 12 comments · Fixed by nextcloud-libraries/nextcloud-dialogs#973
Closed
4 tasks done

📍 File picker dialog improvements #38926

jancborchardt opened this issue Jun 21, 2023 · 12 comments · Fixed by nextcloud-libraries/nextcloud-dialogs#973
Assignees
Labels
1. to develop Accepted and waiting to be taken care of design Design, UI, UX, etc. enhancement feature: filepicker overview

Comments

@jancborchardt
Copy link
Member

jancborchardt commented Jun 21, 2023

How to use GitHub

  • Please use the 👍 reaction to show that you are interested into the same feature.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Is your feature request related to a problem? Please describe.
The current file picker is very basic at the moment – after talking with @jospoortvliet about it we identified some improvements:

Describe the solution you'd like

What do you think @nextcloud/designers – any additional points, and who is interested in doing mockups for it? :)

@jancborchardt jancborchardt added enhancement design Design, UI, UX, etc. 1. to develop Accepted and waiting to be taken care of feature: filepicker labels Jun 21, 2023
@kesselb
Copy link
Contributor

kesselb commented Jun 21, 2023

Remember last location/folder you were in, not start in root folder every time

#644 time goes by so fast ;)

@AndyScherzinger
Copy link
Member

@jancborchardt since we also had this issue in the main view a while ago:

Ability to search in the picker

  • 🔎_Search_ or 🌪️_Filter_ (files currently does both, first filters current folder listing, yet searches everywhere in parallel)
  • Where would you search? In the folder you are currently in? "Everywhere" files related? (bullet point (1) might implicitly answer it dep4ending on your answer to it)

@jancborchardt
Copy link
Member Author

@AndyScherzinger we could start with Filter in current view rather than search. But also, out of the things listed, that is the least priority issue (also cause probably needs a lot of investment).

@nimishavijay
Copy link
Member

Will pick this up for mockups!

Additionally, it would be great to allow sharing via link from the file picker itself wherever appropriate. Related issue: nextcloud/mail#6080 what do you think?

@nimishavijay nimishavijay self-assigned this Jun 26, 2023
@jancborchardt
Copy link
Member Author

Additionally, it would be great to allow sharing via link from the file picker itself wherever appropriate. Related issue: nextcloud/mail#6080 what do you think?

@nimishavijay sounds good as an additional optional mockup, yep! :)

@AndyScherzinger
Copy link
Member

please align any specific @nimishavijay and @susnux - I guess sorting and remembering the last location are clear - however I would challenge the sorting part, but will do in the respective ticket.

@AndyScherzinger AndyScherzinger changed the title File picker dialog improvements 📍 File picker dialog improvements Jul 6, 2023
@nimishavijay
Copy link
Member

nimishavijay commented Jul 12, 2023

@susnux The new file picker can look like this:

image
image

Nice-to-haves

image
image

  • Better UI for attaching items from File mail#6080 Wherever applicable, there could be the option to share a link from within the file picker. Currently, for eg. in Mail, you have the option to choose that you want to share a link, and then the file picker opens, while this flow can be simplified by just having the option to share link from within the file picker anyway.

image

@AndyScherzinger
Copy link
Member

@nimishavijay @jancborchardt just since it got phrased differently in the latest mock up and description: Search should just be filtering for the moment and filtering would be on the current listing, no actual search on all your accessible files.

@nimishavijay
Copy link
Member

@AndyScherzinger understood. then to indicate that search has been applied only on the current listing, the Search results for "design" heading can be under the breadcrumbs bar in the all files view and in the heading with a dot separator in recent and favorites (Recent items · Search results for "design"). cc @susnux and @nextcloud/designers

@marcoambrosini
Copy link
Member

@nimishavijay I would strongly recommend to have the search-box on top of the search results like in some of your other mockups. I find it very counterintuitive that the search-box is filtering a list that it's not directly below it.

If the search-box was directly above the results, I think that there would be less need for a heading indicating that a filter is active, and it would be enough to have a search in "design" placeholder within the search input.

@susnux
Copy link
Contributor

susnux commented Jul 18, 2023

I like the design and it looks consistent :)
But for mobile I am not sure how this should look like. Should the left side menu simply be moved as a horizontal menu underneath the title (between title and current path)?

Because for the app settings, the left side simply is removed.

@nimishavijay
Copy link
Member

But for mobile I am not sure how this should look like. Should the left side menu simply be moved as a horizontal menu underneath the title (between title and current path)?

As discussed yesterday it can be a dropdown item which is shown above the breadcrumbs bar :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of design Design, UI, UX, etc. enhancement feature: filepicker overview
Projects
Archived in project
Archived in project
Development

Successfully merging a pull request may close this issue.

8 participants