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

Issue 29452268: No Issue - CSS Media Query Packer Proposal (Closed)

Created:
May 30, 2017, 4:24 p.m. by ire
Modified:
July 14, 2017, 2:28 p.m.
Visibility:
Public.

Description

No Issue - CSS Media Query Packer Proposal

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -0 lines) Patch
A README.md View 1 chunk +18 lines, -0 lines 0 comments Download

Messages

Total messages: 5
ire
May 30, 2017, 4:24 p.m. (2017-05-30 16:24:12 UTC) #1
juliandoucette
Not LGTM I don't have a strong opinion on this one. But I don't think ...
May 31, 2017, 2:05 p.m. (2017-05-31 14:05:28 UTC) #2
juliandoucette
@Thomas Same here, can you please approve/disapprove so that we can close this.
July 14, 2017, 12:17 p.m. (2017-07-14 12:17:47 UTC) #3
Thomas Greiner
On 2017/07/14 12:17:47, juliandoucette wrote: > @Thomas Same here, can you please approve/disapprove so that ...
July 14, 2017, 12:31 p.m. (2017-07-14 12:31:47 UTC) #4
ire
July 14, 2017, 2:28 p.m. (2017-07-14 14:28:39 UTC) #5
On 2017/07/14 12:31:47, Thomas Greiner wrote:
> Generally, I'd consider it to be sufficient for one reviewer to reject a
review

Noted. Closing this review.

Powered by Google App Engine
This is Rietveld