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

Issue 29555849: Issue 5778 - Hide More filters section when there is no filter (Closed)

Created:
Sept. 25, 2017, 8:10 p.m. by saroyanm
Modified:
Sept. 28, 2017, 11:24 a.m.
Reviewers:
ire
CC:
Thomas Greiner
Visibility:
Public.

Description

Issue 5778 - Hide More filters section when there is no filter

Patch Set 1 #

Total comments: 4

Patch Set 2 : Rebased #

Patch Set 3 : Fixed nits #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -17 lines) Patch
M new-options.html View 1 chunk +1 line, -1 line 0 comments Download
M new-options.js View 1 2 7 chunks +28 lines, -16 lines 0 comments Download

Messages

Total messages: 4
saroyanm
@Ire can you please have a look
Sept. 25, 2017, 8:16 p.m. (2017-09-25 20:16:05 UTC) #1
ire
A couple of NITs, but otherwise this LGTM https://codereview.adblockplus.org/29555849/diff/29555850/new-options.js File new-options.js (right): https://codereview.adblockplus.org/29555849/diff/29555850/new-options.js#newcode55 new-options.js:55: Collection.prototype._setEmpty ...
Sept. 26, 2017, 8:37 a.m. (2017-09-26 08:37:55 UTC) #2
saroyanm
Fixed the nits https://codereview.adblockplus.org/29555849/diff/29555850/new-options.js File new-options.js (right): https://codereview.adblockplus.org/29555849/diff/29555850/new-options.js#newcode55 new-options.js:55: Collection.prototype._setEmpty = function(table, detail, removeEmpty) On ...
Sept. 26, 2017, 4:35 p.m. (2017-09-26 16:35:26 UTC) #3
ire
Sept. 28, 2017, 7:09 a.m. (2017-09-28 07:09:47 UTC) #4
On 2017/09/26 16:35:26, saroyanm wrote:
> Fixed the nits

Thanks :)

Powered by Google App Engine
This is Rietveld