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

Issue 29539750: Issue 5646 - Added Scottish flag (Closed)

Created:
Sept. 8, 2017, 2:11 p.m. by wspee
Modified:
Sept. 12, 2017, 2:36 p.m.
Reviewers:
ire, juliandoucette
Visibility:
Public.

Description

Issue 5646 - Added Scottish flag

Patch Set 1 #

Patch Set 2 : Renamed existing flags to remain consistent #

Patch Set 3 : Changed flag names & README.md #

Patch Set 4 : Renamed the uk flag #

Patch Set 5 : Fixed wrong flag names in globals.tmpl #

Unified diffs Side-by-side diffs Delta from patch set Stats (+92 lines, -92 lines) Patch
M includes/globals.tmpl View 1 2 3 4 1 chunk +89 lines, -89 lines 0 comments Download
M includes/jobs/header.tmpl View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M static/images/flags/README.txt View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
D static/images/flags/am.png View 1 Binary file 0 comments Download
A + static/images/flags/armenia.png View 1 Binary file 0 comments Download
D static/images/flags/at.png View 1 Binary file 0 comments Download
A + static/images/flags/austria.png View 1 Binary file 0 comments Download
D static/images/flags/ba.png View 1 Binary file 0 comments Download
D static/images/flags/bg.png View 1 Binary file 0 comments Download
A + static/images/flags/bosnia-herzegovina.png View 1 2 Binary file 0 comments Download
D static/images/flags/br.png View 1 Binary file 0 comments Download
A + static/images/flags/brazil.png View 1 Binary file 0 comments Download
A + static/images/flags/bulgaria.png View 1 Binary file 0 comments Download
D static/images/flags/ca.png View 1 Binary file 0 comments Download
A + static/images/flags/canada.png View 1 Binary file 0 comments Download
A + static/images/flags/china.png View 1 Binary file 0 comments Download
D static/images/flags/cn.png View 1 Binary file 0 comments Download
D static/images/flags/de.png View 1 Binary file 0 comments Download
D static/images/flags/eg.png View 1 Binary file 0 comments Download
A + static/images/flags/egypt.png View 1 Binary file 0 comments Download
D static/images/flags/es.png View 1 Binary file 0 comments Download
D static/images/flags/fr.png View 1 Binary file 0 comments Download
A + static/images/flags/france.png View 1 Binary file 0 comments Download
D static/images/flags/gb.png View 1 Binary file 0 comments Download
A + static/images/flags/germany.png View 1 Binary file 0 comments Download
D static/images/flags/gr.png View 1 Binary file 0 comments Download
A + static/images/flags/greece.png View 1 Binary file 0 comments Download
D static/images/flags/id.png View 1 Binary file 0 comments Download
D static/images/flags/il.png View 1 Binary file 0 comments Download
D static/images/flags/in.png View 1 Binary file 0 comments Download
A + static/images/flags/india.png View 1 Binary file 0 comments Download
A + static/images/flags/indonesia.png View 1 Binary file 0 comments Download
A + static/images/flags/israel.png View 1 Binary file 0 comments Download
D static/images/flags/ke.png View 1 Binary file 0 comments Download
A + static/images/flags/kenya.png View 1 Binary file 0 comments Download
A + static/images/flags/malaysia.png View 1 Binary file 0 comments Download
A + static/images/flags/mexico.png View 1 Binary file 0 comments Download
D static/images/flags/mx.png View 1 Binary file 0 comments Download
D static/images/flags/my.png View 1 Binary file 0 comments Download
A + static/images/flags/netherlands.png View 1 Binary file 0 comments Download
D static/images/flags/ng.png View 1 Binary file 0 comments Download
A + static/images/flags/nigeria.png View 1 Binary file 0 comments Download
D static/images/flags/nl.png View 1 Binary file 0 comments Download
D static/images/flags/no.png View 1 Binary file 0 comments Download
A + static/images/flags/norway.png View 1 Binary file 0 comments Download
A + static/images/flags/pakistan.png View 1 Binary file 0 comments Download
D static/images/flags/pk.png View 1 Binary file 0 comments Download
D static/images/flags/pl.png View 1 Binary file 0 comments Download
A + static/images/flags/poland.png View 1 Binary file 0 comments Download
A + static/images/flags/portugal.png View 1 Binary file 0 comments Download
D static/images/flags/pt.png View 1 Binary file 0 comments Download
D static/images/flags/ro.png View 1 Binary file 0 comments Download
A + static/images/flags/romania.png View 1 Binary file 0 comments Download
D static/images/flags/ru.png View 1 Binary file 0 comments Download
A + static/images/flags/russia.png View 1 Binary file 0 comments Download
A static/images/flags/scotland.png View Binary file 0 comments Download
D static/images/flags/se.png View 1 Binary file 0 comments Download
A + static/images/flags/spain.png View 1 Binary file 0 comments Download
A + static/images/flags/sweden.png View 1 Binary file 0 comments Download
D static/images/flags/ua.png View 1 Binary file 0 comments Download
A + static/images/flags/ukraine.png View 1 Binary file 0 comments Download
A + static/images/flags/united-kingdom.png View 1 2 3 Binary file 0 comments Download
A + static/images/flags/united-states.png View 1 2 Binary file 0 comments Download
D static/images/flags/us.png View 1 Binary file 0 comments Download
A + static/images/flags/vietnam.png View 1 2 Binary file 0 comments Download
D static/images/flags/vn.png View 1 Binary file 0 comments Download

Messages

Total messages: 15
wspee
Sept. 8, 2017, 2:13 p.m. (2017-09-08 14:13:08 UTC) #1
juliandoucette
LGTM
Sept. 8, 2017, 2:58 p.m. (2017-09-08 14:58:58 UTC) #2
wspee
On 2017/09/08 14:58:58, juliandoucette wrote: > LGTM As discussed I have update the review to ...
Sept. 8, 2017, 3:40 p.m. (2017-09-08 15:40:52 UTC) #3
juliandoucette
NIT: - bosnia-and-herzegovina -> bosnia-herzegovina (opinionated) - reat-britain -> united-kingdom or uk (opinionated + their ...
Sept. 8, 2017, 5:14 p.m. (2017-09-08 17:14:28 UTC) #4
juliandoucette
NIT: - why is Vietnam hyphenated?
Sept. 8, 2017, 5:16 p.m. (2017-09-08 17:16:15 UTC) #5
juliandoucette
Note: - I don't know what I'm talking about regarding UK flags. I based my ...
Sept. 8, 2017, 5:22 p.m. (2017-09-08 17:22:47 UTC) #6
ire
Thanks Winsley! Could you please also update the [README](https://hg.adblockplus.org/web.eyeo.com/file/tip/static/images/flags/README.txt) on how to add new flags ...
Sept. 11, 2017, 7:49 a.m. (2017-09-11 07:49:35 UTC) #7
wspee
On 2017/09/08 17:14:28, juliandoucette wrote: > NIT: > - bosnia-and-herzegovina -> bosnia-herzegovina (opinionated) > - ...
Sept. 11, 2017, 8:27 a.m. (2017-09-11 08:27:41 UTC) #8
wspee
On 2017/09/08 17:16:15, juliandoucette wrote: > NIT: > - why is Vietnam hyphenated? Done
Sept. 11, 2017, 8:28 a.m. (2017-09-11 08:28:05 UTC) #9
wspee
On 2017/09/11 07:49:35, ire wrote: > Thanks Winsley! > > Could you please also update ...
Sept. 11, 2017, 8:28 a.m. (2017-09-11 08:28:28 UTC) #10
ire
On 2017/09/11 08:28:28, wspee wrote: > On 2017/09/11 07:49:35, ire wrote: > > Thanks Winsley! ...
Sept. 11, 2017, 9:43 a.m. (2017-09-11 09:43:47 UTC) #11
juliandoucette
On 2017/09/11 08:27:41, wspee wrote: > I agree about the long names. It seems Great ...
Sept. 11, 2017, 2:36 p.m. (2017-09-11 14:36:20 UTC) #12
wspee
On 2017/09/11 14:36:20, juliandoucette wrote: > On 2017/09/11 08:27:41, wspee wrote: > > I agree ...
Sept. 11, 2017, 2:45 p.m. (2017-09-11 14:45:19 UTC) #13
juliandoucette
LGTM
Sept. 11, 2017, 2:54 p.m. (2017-09-11 14:54:13 UTC) #14
juliandoucette
Sept. 12, 2017, 1:23 p.m. (2017-09-12 13:23:39 UTC) #15
Pushed. Please close.

Powered by Google App Engine
This is Rietveld