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

Issue 29335186: Issue 3025 - Adjusting content margin and font size for small screens in web.adblockbrowser.org (Closed)

Created:
Feb. 1, 2016, 5:12 p.m. by juliandoucette
Modified:
Feb. 10, 2016, 5:33 p.m.
Reviewers:
saroyanm
CC:
Thomas Greiner
Visibility:
Public.

Description

Issue 3025 - Adjusting content margin and font size for small screens in web.adblockbrowser.org

Patch Set 1 #

Total comments: 4

Patch Set 2 : Updated max width and removed font size change #

Patch Set 3 : Removed unused padding #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -3 lines) Patch
M static/css/main.css View 1 2 2 chunks +2 lines, -3 lines 0 comments Download

Messages

Total messages: 4
juliandoucette
Feb. 1, 2016, 5:12 p.m. (2016-02-01 17:12:58 UTC) #1
saroyanm
https://codereview.adblockplus.org/29335186/diff/29335187/static/css/main.css File static/css/main.css (right): https://codereview.adblockplus.org/29335186/diff/29335187/static/css/main.css#newcode585 static/css/main.css:585: max-width: 88%; Can't see a reason why this was ...
Feb. 10, 2016, 2:45 p.m. (2016-02-10 14:45:47 UTC) #2
juliandoucette
https://codereview.adblockplus.org/29335186/diff/29335187/static/css/main.css File static/css/main.css (right): https://codereview.adblockplus.org/29335186/diff/29335187/static/css/main.css#newcode585 static/css/main.css:585: max-width: 88%; On 2016/02/10 14:45:47, saroyanm wrote: > Can't ...
Feb. 10, 2016, 4:58 p.m. (2016-02-10 16:58:38 UTC) #3
saroyanm
Feb. 10, 2016, 5:23 p.m. (2016-02-10 17:23:49 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld