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

Issue 6038021765857280: Issue 1650 - Added disclaimer to homepage for French users (Infrastructure) (Closed)

Created:
Dec. 3, 2014, 11:34 a.m. by Thomas Greiner
Modified:
Dec. 3, 2014, 2:43 p.m.
Visibility:
Public.

Description

See http://codereview.adblockplus.org/6296170825515008/ for Websites part.

Patch Set 1 #

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

Messages

Total messages: 3
Thomas Greiner
Dec. 3, 2014, 11:39 a.m. (2014-12-03 11:39:11 UTC) #1
Wladimir Palant
I didn't review the style changes in detail but the approach looks fine.
Dec. 3, 2014, 1:48 p.m. (2014-12-03 13:48:26 UTC) #2
saroyanm
Dec. 3, 2014, 2:22 p.m. (2014-12-03 14:22:00 UTC) #3
On 2014/12/03 13:48:26, Wladimir Palant wrote:
> I didn't review the style changes in detail but the approach looks fine.

LGTM

Powered by Google App Engine
This is Rietveld