nextcloud / assistant

✨ Nextcloud Assistant
GNU Affero General Public License v3.0
23 stars 4 forks source link

Not enough space in the header with assistant enabled #33

Open juliushaertl opened 7 months ago

juliushaertl commented 7 months ago

On mobile devices the menu bar is too crowded with the assistant enabled

Screenshot 2024-01-26 at 14 44 19

While this is probably not that easy to solve just in the assistant app I though it is the best place to bring this up here for discussion with @nextcloud/designers

marcoambrosini commented 7 months ago

I suggest to:

mwinkens commented 7 months ago

Possible workaround: Install the custom menu app, since the navigation is moved into sidebar/top bar

jancborchardt commented 6 months ago
  • Remove logo on screens that are so narrow;

Not sure about this one – we want the recognition, so removing the logo in some cases would be odd.

  • Assistant can be grouped with other apps or put in the user menu on screens so narrow. The case could be made that it is an app too, so it should maybe always go with other apps?

This sounds reasonable, but I wouldn’t make it part of the app list since that can be ellipsized quite early. However at a certain breakpoint (as soon as we have the right header part overlapping) it could be put into the apps overflow menu. Would be good to have as the first item there, so it’s findable still. Would that be feasible @juliushaertl @julien-nc?

  • People search should be removed altogether, as it duplicates functionality that's already available in the unified search. I've argued for this here. Let's make it easier to search and interact with people in the unified search, instead of having a stripped down version of it right next to it?

As explained in https://github.com/nextcloud/server/issues/41790#issuecomment-1896224847

the People menu is not just "Contacts search". It aims to connect you to colleagues and other people on the instance that you work with a lot. We should improve it rather than remove it.

For example by sorting people up top who you interact with a lot, or people on your team, etc. Rather than advocating for getting rid of it and introducing a regression, can you think of improvements in that direction @marcoambrosini? There’s plenty more places where we can also use more rich "people" display, rather than removing it, for example a better hovercard, displaying info of a person next to a mail from them, or in the chat somewhere. Just some ideas, also cc @nextcloud/designers

susnux commented 6 months ago

Maybe also add a overflow menu for right-side-navigation-entries?

Just like with apps we could have a second action menu where those actions are placed if the space is not enough (for apps we have the ... menu)

jancborchardt commented 6 months ago

@susnux a separate overflow menu for the right is a bit overkill. What do you think about my above proposal?

At a certain breakpoint (as soon as we would have the right header part overlapping) the assistant could be put into the apps overflow menu. Would be good to have as the first item there, so it’s findable still.