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

Issue 29834609: Fixes #108 - Moved video disclaimer underneath video on small screens and removed it's show/hide an… (Closed)

Created:
July 19, 2018, 5:05 p.m. by juliandoucette
Modified:
July 21, 2018, 2:26 a.m.
Reviewers:
ire
CC:
saroyanm
Base URL:
https://hg.adblockplus.org/web.adblockplus.org
Visibility:
Public.

Description

Fixes #108 - Moved video disclaimer underneath video on small screens and removed it's show/hide animation

Patch Set 1 #

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

Messages

Total messages: 4
juliandoucette
July 19, 2018, 5:05 p.m. (2018-07-19 17:05:14 UTC) #1
juliandoucette
Relevant: https://gitlab.com/eyeo/websites/web.adblockplus.org/issues/108#note_89153618
July 19, 2018, 7:15 p.m. (2018-07-19 19:15:16 UTC) #2
juliandoucette
Andrea looked at this for me. I think that my eyes and his are enough ...
July 20, 2018, 8:24 p.m. (2018-07-20 20:24:31 UTC) #3
ire
July 21, 2018, 2:26 a.m. (2018-07-21 02:26:56 UTC) #4
Message was sent while issue was closed.
On 2018/07/20 20:24:31, juliandoucette wrote:
> Andrea looked at this for me. I think that my eyes and his are enough given
the
> size of this change.

Ack. LGTM FWIW :)

Powered by Google App Engine
This is Rietveld