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

Issue 29721720: Fixes #32 - Adjustments to cross-promotion section (Closed)

Created:
March 13, 2018, 4:55 p.m. by juliandoucette
Modified:
March 14, 2018, 2:58 p.m.
Reviewers:
ire
Base URL:
https://hg.adblockplus.org/web.adblockplus.org
Visibility:
Public.

Description

Please see https://gitlab.com/eyeo/websites/web.adblockplus.org/issues/32#note_63026898 before reviewing.

Patch Set 1 #

Patch Set 2 : Addressed note_63196626 #

Patch Set 3 : Rebased #

Patch Set 4 : Removed global font-size change #

Unified diffs Side-by-side diffs Delta from patch set Stats (+32 lines, -3 lines) Patch
M includes/cross-promotion.tmpl View 1 chunk +1 line, -1 line 0 comments Download
M locales/en/abb-notification.json View 1 chunk +1 line, -1 line 0 comments Download
M static/css/index.css View 1 2 1 chunk +30 lines, -1 line 0 comments Download

Messages

Total messages: 9
juliandoucette
March 13, 2018, 4:55 p.m. (2018-03-13 16:55:46 UTC) #1
juliandoucette
Please see https://gitlab.com/eyeo/websites/web.adblockplus.org/issues/32#note_63026898 before reviewing.
March 13, 2018, 4:56 p.m. (2018-03-13 16:56:02 UTC) #2
ire
On 2018/03/13 16:56:02, juliandoucette wrote: > Please see > https://gitlab.com/eyeo/websites/web.adblockplus.org/issues/32#note_63026898 > before reviewing. Ack
March 14, 2018, 7:57 a.m. (2018-03-14 07:57:47 UTC) #3
juliandoucette
New Patchset up! (It's ready to be reviewed now) Detail: I added another layer of ...
March 14, 2018, 12:27 p.m. (2018-03-14 12:27:33 UTC) #4
juliandoucette
On 2018/03/14 12:27:33, juliandoucette wrote: > Detail: I added another layer of responsiveness to heading ...
March 14, 2018, 12:48 p.m. (2018-03-14 12:48:34 UTC) #5
ire
@jeen: > @juliandoucette for the phones I would reduce the line height by 0.5em and ...
March 14, 2018, 1:17 p.m. (2018-03-14 13:17:11 UTC) #6
juliandoucette
On 2018/03/14 13:17:11, ire wrote: > You didn't address this? See my last two replies ...
March 14, 2018, 1:55 p.m. (2018-03-14 13:55:46 UTC) #7
ire
On 2018/03/14 13:55:46, juliandoucette wrote: > On 2018/03/14 13:17:11, ire wrote: > > You didn't ...
March 14, 2018, 2:12 p.m. (2018-03-14 14:12:39 UTC) #8
juliandoucette
March 14, 2018, 2:58 p.m. (2018-03-14 14:58:40 UTC) #9

Powered by Google App Engine
This is Rietveld