Closed ghost closed 5 years ago
You forgot to link to the issue. https://github.com/sugarlabs/sugar/issues/797
Reviewed 233515a.
In my opinion, none of these accounts are used by developers who work on Sugar itself; that is the set of people who have made commits to this repository does not contain any of these account owners. It would be a misrepresentation.
That said;
Thanks!
@samswag Can the marketing team please review these links? thx.
@quozl This looks not so good. Any better solution?
The appearance quality is fine. It is a README.md file, after all. The format and syntax is intentionally restricted. You might add a line break after the link to www.sugarlabs.org.
But this does not address my primary concerns.
Yes the links are correct @walterbender.
@quozl @samswag Can anybody review this?
Reviewed to c3a0702. I've no issues with layout, it looks fine.
Sorry, we can't merge those icons into the software package because they don't have a compatible license. A "make dist" build step would add them to the release tar.xz file, and then we couldn't distribute the file.
Please move the icons to the www-sugarlabs repository and then use the links to them on our web server.
Please use a new pull request. I'm closing this one because I don't want these commercial icons to accidentally end up in the repository, which they would if the pull request was merged at this point.
Let me know if you have any questions. I look forward to the next pull request! :grinning:
I got it from here and its is under MIT license which is free to use and distribute. https://github.com/konsav/social-icons
@drashti4, thanks.
@walterbender, does Sugar Labs need a license to use these trademarks? :grin: I've just never come across anyone asking to add trademarked commercial logos to open source software that aren't part of the software in any way.
I am against adding this to the ReadMe, since it's not useful for the intended audience of the ReadMe
Thanks. I agree.
Added icon according to website