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

Issue 4907991358767104: Issue 1816 - Add download Maxthon notification for Chinese locales (Infrastructure) (Closed)

Created:
Jan. 30, 2015, 11:24 a.m. by saroyanm
Modified:
Feb. 10, 2015, 2:26 p.m.
Reviewers:
Thomas Greiner
CC:
Wladimir Palant
Visibility:
Public.

Description

Website changes: http://codereview.adblockplus.org/6553361117609984/ Related ticket: https://issues.adblockplus.org/ticket/1816

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -4 lines) Patch
M anwiki/_override-static/global/global/css/index.css View 1 2 3 chunks +6 lines, -2 lines 0 comments Download
M anwiki/_override-static/global/global/css/index-desktop.css View 1 2 2 chunks +10 lines, -2 lines 0 comments Download
M anwiki/_override-static/global/global/css/index-mobile.css View 1 2 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4
saroyanm
@Thomas while there is only CSS changes I've added you as reviewer, please have a ...
Jan. 30, 2015, 11:48 a.m. (2015-01-30 11:48:43 UTC) #1
saroyanm
On 2015/01/30 11:48:43, saroyanm wrote: > @Thomas while there is only CSS changes I've added ...
Feb. 6, 2015, 10:41 a.m. (2015-02-06 10:41:12 UTC) #2
saroyanm
I've updated current review according to the last changes in related "website changes" ticket, also ...
Feb. 9, 2015, 11:23 a.m. (2015-02-09 11:23:50 UTC) #3
Thomas Greiner
Feb. 9, 2015, 11:29 a.m. (2015-02-09 11:29:26 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld