Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 5097747367591936: Issue 332 - Add country flags to team page (Closed)

Created:
April 29, 2014, 4:29 p.m. by Thomas Greiner
Modified:
May 2, 2014, 1:24 p.m.
Visibility:
Public.

Description

Issue 332 - Add country flags to team page

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+46 lines, -25 lines) Patch
M includes/globals.tmpl View 1 chunk +23 lines, -23 lines 0 comments Download
M pages/team.tmpl View 2 chunks +4 lines, -1 line 3 comments Download
M static/css/styles.css View 2 chunks +11 lines, -1 line 0 comments Download
A static/images/flags/README.txt View 1 chunk +8 lines, -0 lines 0 comments Download
A static/images/flags/am.png View Binary file 0 comments Download
A static/images/flags/at.png View Binary file 0 comments Download
A static/images/flags/de.png View Binary file 0 comments Download
A static/images/flags/es.png View Binary file 0 comments Download
A static/images/flags/nl.png View Binary file 0 comments Download
A static/images/flags/pl.png View Binary file 0 comments Download
A static/images/flags/sk.png View Binary file 0 comments Download
A static/images/flags/ua.png View Binary file 0 comments Download
A static/images/flags/us.png View Binary file 0 comments Download

Messages

Total messages: 6
Thomas Greiner
In the future we might want to add one single sprite image for all images ...
April 29, 2014, 4:38 p.m. (2014-04-29 16:38:50 UTC) #1
Wladimir Palant
http://codereview.adblockplus.org/5097747367591936/diff/5629499534213120/pages/team.tmpl File pages/team.tmpl (right): http://codereview.adblockplus.org/5097747367591936/diff/5629499534213120/pages/team.tmpl#newcode32 pages/team.tmpl:32: <!-- Flag icons: Copyright (c) 2013 Go Squared Ltd. ...
April 29, 2014, 6:19 p.m. (2014-04-29 18:19:52 UTC) #2
Thomas Greiner
http://codereview.adblockplus.org/5097747367591936/diff/5629499534213120/pages/team.tmpl File pages/team.tmpl (right): http://codereview.adblockplus.org/5097747367591936/diff/5629499534213120/pages/team.tmpl#newcode32 pages/team.tmpl:32: <!-- Flag icons: Copyright (c) 2013 Go Squared Ltd. ...
April 30, 2014, 2:13 p.m. (2014-04-30 14:13:59 UTC) #3
Wladimir Palant
Oh well, LGTM
April 30, 2014, 2:17 p.m. (2014-04-30 14:17:48 UTC) #4
Felix Dahlke
LGTM, feel free to change the license notice to fully comply with the terms, but ...
April 30, 2014, 3:04 p.m. (2014-04-30 15:04:31 UTC) #5
Thomas Greiner
May 2, 2014, 9:46 a.m. (2014-05-02 09:46:35 UTC) #6
On 2014/04/30 15:04:31, Felix H. Dahlke wrote:
> LGTM, feel free to change the license notice to fully comply with the terms,
but
> I guess we're fine if Kai specifically OKd this.
> 
>
http://codereview.adblockplus.org/5097747367591936/diff/5629499534213120/page...
> File pages/team.tmpl (right):
> 
>
http://codereview.adblockplus.org/5097747367591936/diff/5629499534213120/page...
> pages/team.tmpl:32: <!-- Flag icons: Copyright (c) 2013 Go Squared Ltd.
> http://www.gosquared.com/ -->
> On 2014/04/30 14:13:59, Thomas Greiner wrote:
> > On 2014/04/29 18:19:53, Wladimir Palant wrote:
> > > "The above copyright notice and this permission notice shall be included
in
> > all
> > > copies or substantial portions of the Software." - I don't think that this
> > line
> > > complies to their license. Aren't there any public domain flag icons where
> we
> > > don't have licensing headaches?
> > 
> > This addition was approved by Kai beforehand although he doesn't consider it
> > absolutely necessary. It definitely doesn't hurt to have it in there but I
can
> > ask the icon creators for clarification on that to make sure.
> > 
> > After putting more time into searching for alternatives, I was not able to
> find
> > flags that fit into the flat-design scheme of our page without looking cheap
> > while not requiring backlinks or payment.
> 
> Why not just do what they ask for and include the permission notice? Doesn't
> seem like something we need to involve a lawyer over. But if Kai already said
he
> thinks it's fine...

FYI: I received the response from the icons' creators. Their only concern is
that someone might redistribute the icons or claim ownership so we are fine.
They say that the license was not meant for the icons but for the buildtools in
https://github.com/gosquared/flags.

Generally, we don't need to include anything but they would appreciate some sort
of attribution so I'll keep that copyright notice in there.

Powered by Google App Engine
This is Rietveld