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

Content list documentation #638

Merged
merged 2 commits into from
Jan 3, 2018
Merged

Content list documentation #638

merged 2 commits into from
Jan 3, 2018

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Dec 21, 2017

After nextcloud/server#7195

Please review!
Is it clear enough? :)

Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>
@jancborchardt
Copy link
Member

Nice! :) Should we specifically say something about that we recommend using the app-content-list-item-icon for profile photos, or favicons, etc?

cc @nextcloud/designers

@skjnldsv
Copy link
Member Author

@jancborchardt yes, it's a requirement :)

@juliusknorr
Copy link
Member

Looks good to me. Just the screenshot besides the first code snippet should be moved above, because it currently shrinks the code block so that it is quite hard to read.

@skjnldsv
Copy link
Member Author

skjnldsv commented Jan 3, 2018

@juliushaertl I'll remove the inline image code block

EDIT: done, please review

Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>
@juliusknorr juliusknorr merged commit f13ea01 into master Jan 3, 2018
@juliusknorr juliusknorr deleted the content-list branch January 3, 2018 15:54
@jancborchardt
Copy link
Member

Aaawesome! :) Great job with the docs as always @skjnldsv

@skjnldsv
Copy link
Member Author

skjnldsv commented Jan 5, 2018

Thank Jan! 🤗

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

Successfully merging this pull request may close these issues.

3 participants