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

Issue 29601607: Issue 5963 - Fix alignment of product icon next to article title on help.eyeo.com (Closed)

Created:
Nov. 8, 2017, 3:37 p.m. by ire
Modified:
Nov. 10, 2017, 12:03 p.m.
Reviewers:
juliandoucette
Base URL:
https://hg.adblockplus.org/help.eyeo.com
Visibility:
Public.

Description

Issue 5963 - Fix alignment of product icon next to article title on help.eyeo.com

Patch Set 1 #

Total comments: 2

Patch Set 2 : max-width: - 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -0 lines) Patch
M static/scss/components/_article.scss View 1 1 chunk +20 lines, -0 lines 0 comments Download

Messages

Total messages: 4
ire
Nov. 8, 2017, 3:37 p.m. (2017-11-08 15:37:17 UTC) #1
juliandoucette
https://codereview.adblockplus.org/29601607/diff/29601608/static/scss/components/_article.scss File static/scss/components/_article.scss (right): https://codereview.adblockplus.org/29601607/diff/29601608/static/scss/components/_article.scss#newcode41 static/scss/components/_article.scss:41: @media (max-width: $phablet-breakpoint + 1) +1 should be -1. ...
Nov. 9, 2017, 2:22 p.m. (2017-11-09 14:22:30 UTC) #2
ire
https://codereview.adblockplus.org/29601607/diff/29601608/static/scss/components/_article.scss File static/scss/components/_article.scss (right): https://codereview.adblockplus.org/29601607/diff/29601608/static/scss/components/_article.scss#newcode41 static/scss/components/_article.scss:41: @media (max-width: $phablet-breakpoint + 1) On 2017/11/09 14:22:30, juliandoucette ...
Nov. 10, 2017, 9:44 a.m. (2017-11-10 09:44:25 UTC) #3
juliandoucette
Nov. 10, 2017, 10:44 a.m. (2017-11-10 10:44:51 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld