-
Notifications
You must be signed in to change notification settings - Fork 46
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
Add screenshot and fix title #215
Conversation
Title contains now "ownCloud".
By analyzing the blame information on this pull request, we identified @owncloud-news-gitter-bot, @BernhardPosselt and @Henni to be potential reviewers |
There is a small strange part of a circle in the bottom left of the screenshot, can you whiten that? :) Otherwise 👍 |
@stefan-niedermann do you want to take another short look and fix the screenshot? |
Sure, no problem. but how did you imagine? logo and default theme are still pointing to "one of the two" clouds....... |
@stefan-niedermann "Notes" should work for now. So let's leave it like it currently is. This is just about the name. The repository will stay here (owncloud/notes) and we only need one screenshot of the app in one of the clouds. This means that your screenshot is great, except for:
|
@stefan-niedermann yup, no need to add the »ownCloud« in the title as it also runs on Nextcloud. As Gallery, Mail and others have made their apps cloud-independent we should do that with Notes too. @Henni are you at the Meetup this evening? I’ll be in Muc again and @LukasReschke will too :) We can polish the Contacts app more then. |
See #242 - new pull request, only screenshot :) regards |
Title contains now "ownCloud".
#213
https://github.com/stefan-niedermann/screenshots/commit/a2008c461663a943945475cea0d1cebf731bc482