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

Moving Attachments into the first Tab #2691

Closed
jakobroehrl opened this issue Dec 30, 2020 · 5 comments
Closed

Moving Attachments into the first Tab #2691

jakobroehrl opened this issue Dec 30, 2020 · 5 comments

Comments

@jakobroehrl
Copy link
Contributor

grafik

@juliusknorr
Copy link
Member

Please try to provide some arguments for such UI changes in the future and involve @nextcloud/designers for discussion :) (cc-ing @Clementine46 @jancborchardt especially for deck)

I generally don't like putting even more content into the sidebar by default. The tabs are a good way to quickly navigate to that if needed and especially if projects are linked the sidebar becomes rather messed in my opinion with this, so 👎 from my side.

@stefan-niedermann
Copy link
Member

Speaking for the Android app (where we have limited space), i will not follow this approach - actually i planned to do the opposite and move the Project relations into its own tab in the future 😅

@juliusknorr
Copy link
Member

I'd say that is totally fine for limited space (as we also have it in the sidebar). For the modal view see #2702

@jakobroehrl
Copy link
Contributor Author

@stefan-niedermann
I think we should show the attachments on limited space, too. Otherwise it will be overseen.
Example on Trello: (I can provide examples on zenkit, padlet, ... if needed)
grafik

@stefan-niedermann
Copy link
Member

@jakobroehrl while i get the motivation now, i still think that this will lead to a very crowded.
I would prefer to hint to the attachments like proposed in #1635 instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants