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

Issue 29685561: Issue 5864 - Specify CSS group name for standard filters (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 9 months ago by Manish Jethani
Modified:
1 year, 9 months ago
Reviewers:
kzar
CC:
Sebastian Noack
Base URL:
https://hg.adblockplus.org/adblockpluschrome/
Visibility:
Public.

Description

We forgot to do this in https://codereview.adblockplus.org/29575739/

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M lib/cssInjection.js View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4
Manish Jethani
1 year, 9 months ago (2018-01-31 08:31:11 UTC) #1
Manish Jethani
Patch Set 1 https://codereview.adblockplus.org/29575739/
1 year, 9 months ago (2018-01-31 08:32:45 UTC) #2
Manish Jethani
On 2018/01/31 08:32:45, Manish Jethani wrote: > Patch Set 1 > > https://codereview.adblockplus.org/29575739/ I mean ...
1 year, 9 months ago (2018-01-31 08:33:34 UTC) #3
kzar
1 year, 9 months ago (2018-01-31 11:18:50 UTC) #4
You could also give the commit message the issue number of 5864 but I don't care
much either way. This LGTM anyway.
Sign in to reply to this message.

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