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

Issue 29844604: Fixes #115 - Added mobile section to specific criteria of acceptable ads page

Created:
Aug. 1, 2018, 3:02 p.m. by juliandoucette
Modified:
Aug. 16, 2018, 1:45 p.m.
Reviewers:
wspee, ire, l.ursachi
CC:
saroyanm
Base URL:
https://hg.adblockplus.org/web.adblockplus.org
Visibility:
Public.

Description

Fixes #115 - Added mobile section to specific criteria of acceptable ads page

Patch Set 1 #

Total comments: 4

Patch Set 2 : Addressed #2 #

Patch Set 3 : Added animated ads footnote #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -1 line) Patch
M pages/acceptable-ads.md View 1 2 3 chunks +28 lines, -1 line 0 comments Download

Messages

Total messages: 4
juliandoucette
Aug. 1, 2018, 3:02 p.m. (2018-08-01 15:02:42 UTC) #1
l.ursachi
Added a quick and dirty code review - not sure if enough, though :-) https://codereview.adblockplus.org/29844604/diff/29844605/pages/acceptable-ads.md ...
Aug. 3, 2018, 11:31 a.m. (2018-08-03 11:31:43 UTC) #2
wspee
On 2018/08/03 11:31:43, l.ursachi wrote: > Added a quick and dirty code review - not ...
Aug. 3, 2018, 1:40 p.m. (2018-08-03 13:40:32 UTC) #3
juliandoucette
Aug. 16, 2018, 1:38 p.m. (2018-08-16 13:38:10 UTC) #4
Addressed #2

https://codereview.adblockplus.org/29844604/diff/29844605/pages/acceptable-ad...
File pages/acceptable-ads.md (right):

https://codereview.adblockplus.org/29844604/diff/29844605/pages/acceptable-ad...
pages/acceptable-ads.md:94: #### {{ mobile-ads-heading[heading] MOBILE ADS }}
On 2018/08/03 11:31:42, l.ursachi wrote:
> Could we please have ```MOBILE ADS``` in sentence case in order to be
consistent
> with the other H4s? I know it was in upper case in the copy doc but only saw
it
> now :D

Done.

https://codereview.adblockplus.org/29844604/diff/29844605/pages/acceptable-ad...
pages/acceptable-ads.md:98: - {{ mobile-ads-placement-item-1[list item] Static
ad types (e.g. 6x1 banner and 1x1 tile ad) are allowed to be placed anywhere on
the mobile page }}
On 2018/08/03 11:31:42, l.ursachi wrote:
> Add ```.``` at the end of the sentence

Done.

Powered by Google App Engine
This is Rietveld