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

More space for navigation text #2693

Closed
nickvergessen opened this issue Dec 15, 2016 · 18 comments
Closed

More space for navigation text #2693

nickvergessen opened this issue Dec 15, 2016 · 18 comments
Labels
bug design Design, UI, UX, etc.

Comments

@nickvergessen
Copy link
Member

There is really no need to limit the label length to 8 characters:

bildschirmfoto vom 2016-12-15 14-35-40

We have so much space available...

Also the padding left is broken and the text also doesn't look centered:

bildschirmfoto vom 2016-12-15 14-36-17

@nextcloud/designers

@nickvergessen nickvergessen added 1. to develop Accepted and waiting to be taken care of bug design Design, UI, UX, etc. labels Dec 15, 2016
@jancborchardt
Copy link
Member

jancborchardt commented Dec 15, 2016

Of course the limit is necessary, otherwise the names would overflow. Just look at this:
capture du 2016-12-15 15-45-39

The padding issue on the left is correct.

But regarding the actual problem of the long name of »Announcement center«:
I think I mentioned before and I still think that the announcement center should be a part of »Activity«. This adding of apps upon apps doesn’t make sense. Announcements are in the timeline, and they show in the Activity app. The rest to integrate is just putting the input field for admins on the very top of the Activity app. And having a filter on the left for »Announcements« with the announcement icon.
What do you think @nickvergessen?

@jancborchardt jancborchardt removed the 1. to develop Accepted and waiting to be taken care of label Dec 15, 2016
@nickvergessen
Copy link
Member Author

nickvergessen commented Dec 15, 2016

And no, the app does way too much, to "simply" melt it into activities.
This is not what this is about. Please stay on track.

The name is already "Announcements" (if there would at least be hover for ellipsed titles you would see that)
Make it possible that an app can have more then 8 fucking characters as a link

I would also use "News" as a link title, but a well-known "RSS Feed Reader" abuses this name already.

@nickvergessen nickvergessen added this to the Nextcloud 11.0.1 milestone Dec 15, 2016
@jancborchardt
Copy link
Member

And no, the app does way too much, to "simply" melt it into activities.
This is not what this is about. Please stay on track.

Yes it also is about that. Please explain why it does way too much? While code-wise it can stay a different app, interface-wise it can be combined just fine, and should do so. It’s all chronological activity info. Let’s keep things simple here as we do in Nextcloud.

@jancborchardt
Copy link
Member

The name is already "Announcements" (if there would at least be hover for ellipsed titles you would see that)

Yeah, a tooltip for ellipsized titles would be cool. But we need to do that in JS so that not all titles blindly get a hover tooltip. Because most are short enough.

Make it possible that an app can have more then 8 fucking characters as a link

As shown in the screenshot above, how should we do that? Look at your Android phone or an iOS device … there has to be a cutoff at some point. Swearing doesn’t help fix it.

I would also use "News" as a link title, but a well-known "RSS Feed Reader" abuses this name already.

Why abuse? It’s a user-facing app and people read news with it. What does the announcements app have to do with »News«? ;) Sure we can talk about different words like »Notice« or »Post« or whatnot but this tackles the wrong problem. See my previous comment about integration.

@nickvergessen
Copy link
Member Author

What does the announcements app have to do with »News«?

It's news about your instance

I will NEVER integrate this into activities, please stay on track, "Annoucements" is really not a very long word, we have 1600 pixels left on that page,....

@schiessle
Copy link
Member

schiessle commented Dec 16, 2016

I have to agree with @nickvergessen. Also keep translations in mind, some languages use to have longer words than others, some needs 2-3 words for a proper name and some have a catchy short word to name a app. Most of the time you have more than enough space, so why not just let the drop-down grow/shrink with the content and give every app enough space to display the complete app name? I think from a UX it is much nicer to see the complete name than just a few letters followed by '...'

I think the min. requirement is to have a tooltip on hover to show the complete name.

@jancborchardt
Copy link
Member

So we are thinking about using a direct-menu like approach for Nextcloud 12 because also apps are too hidden and not discoverable. Talked with @eppfel @juliushaertl :)

That would also partly take care of this issue.

@schiessle
Copy link
Member

What do you mean by "direct-menu"? Do you already have a mock-up, screenshot, etc? Does this mean to have again a always visible menu like many years ago?

@nickvergessen
Copy link
Member Author

@jancborchardt
Copy link
Member

@schiessle basically similar to this: https://apps.nextcloud.com/apps/direct_menu by @juliushaertl. See discussion at https://help.nextcloud.com/t/is-this-the-most-awesome-or-what/6163/3?u=jan
And yes, always visible but limited to the first ~5 apps (and then a 3-dot-menu for the rest) to not clutter too much.

@schiessle
Copy link
Member

@jancborchardt @nickvergessen thanks for the pointer. To be honest, I'm not a big fan of the "direct-menu" but let's see how things evolve.

@jancborchardt
Copy link
Member

@schiessle it of course needs to be polished, but the problem we need to really solve is people not knowing that Nextcloud has other apps except files. We hear that more and more. :)

@MorrisJobke
Copy link
Member

Fixed by new menu - see #3959 for the remaining issue

@nickvergessen
Copy link
Member Author

This is still a problem in the overview menu.

But jan wanted to change it to a lined layout (like settings menu) instead of the current box-layout.

@jancborchardt is there already a dedicated issue for that?

@nickvergessen nickvergessen reopened this Apr 10, 2017
@MorrisJobke
Copy link
Member

Nothing for 12 I would say.

@eppfel
Copy link
Member

eppfel commented May 15, 2017

So, we have either work out a dynamic solution, which could lead to app names, which are just too long.

Or we agree on a character limit and fix the menu.

In particular changing "announcement center" to "announcements", would solve the issue for now, right?
bildschirmfoto 2017-05-15 um 11 46 56

@jancborchardt
Copy link
Member

But jan wanted to change it to a lined layout (like settings menu) instead of the current box-layout.

This was actually done through @juliushaertl's responsive menu at #4439

The names now have more space, and I would say that’s also enough, no? @nickvergessen what do you think?

EDIT: Yep, as @eppfel said the name should be concise. »Announcements« is good. :)

@nickvergessen
Copy link
Member Author

In particular changing "announcement center" to "announcements"

Hu? the navigation string is already announcements since ever ;)

Anyway, yes this one is solved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug design Design, UI, UX, etc.
Projects
None yet
Development

No branches or pull requests

5 participants