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

Re-order and group drawer items? #13051

Open
joshtrichards opened this issue May 27, 2024 · 4 comments · Fixed by #13610 · May be fixed by #13609
Open

Re-order and group drawer items? #13051

joshtrichards opened this issue May 27, 2024 · 4 comments · Fixed by #13610 · May be fixed by #13609
Assignees
Labels
design enhancement navigation Navigation matters (e.g. covered under #11072)

Comments

@joshtrichards
Copy link
Member

joshtrichards commented May 27, 2024

To better align with the evolving Web UI should we consider tidying up the navigation drawer a bit?

  • splitting up the Files related items from the other ones?
  • adding visual dividers between the logically separate groups of items
  • related:
    • we could maybe also move Activities up to the ecosystem app menu that sits at the top, since it is, in fact an app (and that's consistent with the web UI approach
    • Media isn't exactly the Photos app so it's debatable about moving it there, but that's the only other one that is a candidate
    • Incidentally, maybe there is some way to revisit the design of the ecosystem apps list to make them more compact/list more of them
  • where should Notifications go?

e.g.


Talk | Notes | More

  • All Files
  • Personal Files
  • Favorites
  • Recents <-- (future)
  • Shares <-- "Shared->Shares" w/ own section
  • Tags <-- (future)
  • External Storage <-- (future)

  • Activities
  • Media
  • Uploads
  • On Device

  • Deleted files
  • 559 MB used
  • Settings

Reference: Current Android:

image

Reference: Server v29 Web Client UI:

image

@joshtrichards joshtrichards added enhancement design navigation Navigation matters (e.g. covered under #11072) labels May 27, 2024
@AndyScherzinger
Copy link
Member

looping in @nextcloud/designers for input and feedback

@jancborchardt
Copy link
Member

jancborchardt commented Aug 27, 2024

@joshtrichards liking your suggestions a lot! :)

  • I tend to agree with moving Media up to the integrations section as if it’s Photos
  • Activity I’m not sure about, since it’s similar to "Recents" (which we don’t have yet) and not used so often so doesn’t need to be that present
  • I like the organization / splitting via separator → @AndyScherzinger said this probably has to be whitespace because of Material Design standards
  • I agree that Notifications is too difficult to reach. If there are notifications, can’t we show a notification bell in the header, e.g. left of the avatar in the search field?

@tobiasKaminsky what do you think?

@tobiasKaminsky tobiasKaminsky self-assigned this Sep 9, 2024
@tobiasKaminsky
Copy link
Member

2024-09-24-084530

What do you think? There are some apps, e.g. groupfolder which are not always enabled. I tried to keep them either in first group (files) or second (rest).

Notification:
When we receive push notifications, then we can show a red bell/dot somewhere.
But then the user already sees it on OS level.

This notification section serves mainly two things

  • if push notification was closed, but not dismissed, then user can find it there
  • if no push notification is available, e.g. on Fdroid/huawei/… -> then this is the only way to find it

So in first case we could show it, but usually user directly interact with push notifications (e.g. grant access via it).
In second case (and also partly for first case) we could always ask on app start to fetch notification count.

  • If there are notifications, can’t we show a notification bell in the header, e.g. left of the avatar in the search field?

We can, if we check.

@joshtrichards
Copy link
Member Author

👍
Auto-closed w/ the notifications PR merge (I think).
Re-opened to not lose track of the other associated work, namely PR #13609.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design enhancement navigation Navigation matters (e.g. covered under #11072)
Projects
Status: 🏗️ At engineering
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants