Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(306)

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 months, 2 weeks ago by juliandoucette
Modified:
2 months ago
Reviewers:
l.ursachi, wspee, ire
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
2 months, 2 weeks ago (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 ...
2 months, 2 weeks ago (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 ...
2 months, 2 weeks ago (2018-08-03 13:40:32 UTC) #3
juliandoucette
2 months ago (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.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 87257f5