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

Issue 29348926: Issue 4241 - Add web.adblockbrowser.org to https://adblockplus.org/modules (Closed)

Created:
Aug. 1, 2016, 1:48 p.m. by juliandoucette
Modified:
Aug. 15, 2016, 6:38 p.m.
Reviewers:
saroyanm
Visibility:
Public.

Description

Issue 4241 - Add web.adblockbrowser.org to https://adblockplus.org/modules

Patch Set 1 #

Patch Set 2 : Re-ordered modules #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -5 lines) Patch
M pages/modules.html View 1 1 chunk +11 lines, -5 lines 0 comments Download

Messages

Total messages: 6
juliandoucette
Aug. 1, 2016, 1:48 p.m. (2016-08-01 13:48:07 UTC) #1
juliandoucette
Should be a quick one. I changed the order of modules because I think it ...
Aug. 1, 2016, 1:50 p.m. (2016-08-01 13:50:39 UTC) #2
saroyanm
On 2016/08/01 13:50:39, juliandoucette wrote: > Should be a quick one. > > I changed ...
Aug. 1, 2016, 1:54 p.m. (2016-08-01 13:54:19 UTC) #3
juliandoucette
On 2016/08/01 13:54:19, saroyanm wrote: > On 2016/08/01 13:50:39, juliandoucette wrote: > > Should be ...
Aug. 1, 2016, 2:22 p.m. (2016-08-01 14:22:21 UTC) #4
saroyanm
On 2016/08/01 14:22:21, juliandoucette wrote: > On 2016/08/01 13:54:19, saroyanm wrote: > > On 2016/08/01 ...
Aug. 2, 2016, 5:21 p.m. (2016-08-02 17:21:27 UTC) #5
juliandoucette
Aug. 15, 2016, 6:38 p.m. (2016-08-15 18:38:07 UTC) #6
> Actually by "others" I meant also http://share.adblockplus.org, 
> facebook.adblockplus.me and etc. 
> They are not the important one so they can be below. 
> With the comment above - LGTM

Agreed. I pushed the order you suggested.

- https://hg.adblockplus.org/web.adblockplus.org/rev/aa703d90c34f

Powered by Google App Engine
This is Rietveld