Calendar: Shared calendar entry to crowded in the navigation
I guess the "Shared" text could just be omitted:
cc @nextcloud/designers
juliushaertl
All 5 comments
Another option we have would be switching to the AppContentListItem component that we'll be using in talk, contacts and mail. That way we'd have double the height to organize all those elements.
ma12-co
on 11 Nov 2019
@ma12-co The item is in the AppNavigation, not AppContentList :)
georgehrke
on 11 Nov 2019
I'm fine removing it.
We already show the shared state with the opacity.
The idea was to have it consistently with Files, where you also have that label. But in the file-list there is obviously much more space, compared to AppNavigation.
georgehrke
on 11 Nov 2019
@ma12-co The item is in the AppNavigation, not AppContentList :)
Yes, but Marco built a standard component that works in both locations (as same design and layout for mail/contacts/talk) :+1:
skjnldsv
on 12 Nov 2019
We do have the 2-row component of course, but why should we use it for the Calendar here? :) It's for different use-cases where much more info is displayed.
If you look at Apple Calendar, Google Calendar etc, they have a very similar layout for the list as we have currently.
We shouldn't use components for the sake of using components. The components follow what makes sense design-wise.
In this case, removing the "Shared" text as @juliushaertl mentioned is already enough. We could also additionally see to combine the link icon and the avatar, but that's something separate.
jancborchardt
on 12 Nov 2019
Related issues
brylie
路
3Comments
deanforrest
路
4Comments
skjnldsv
路
4Comments
cloud2018
路
3Comments
scorewinner
路
3Comments
Most helpful comment
I'm fine removing it.
We already show the shared state with the opacity.
The idea was to have it consistently with Files, where you also have that label. But in the file-list there is obviously much more space, compared to AppNavigation.