5 years, 10 months ago
(2015-02-19 09:56:26 UTC)
#2
Could you take a look? Thanks!
yawano
Just one question. Who uses expand_less.png?
5 years, 10 months ago
(2015-02-19 10:26:00 UTC)
#3
Just one question. Who uses expand_less.png?
fukino
On 2015/02/19 10:26:00, yawano wrote: > Just one question. Who uses expand_less.png? expand_less.png is not ...
5 years, 10 months ago
(2015-02-19 10:33:17 UTC)
#4
On 2015/02/19 10:26:00, yawano wrote:
> Just one question. Who uses expand_less.png?
expand_less.png is not in use. Should I remove it?
I thought, as we use expand_more, expand_more_active, and expand_less_active,
absence of expand_less might be confusing. WDYT?
yawano
On 2015/02/19 10:33:17, fukino wrote: > On 2015/02/19 10:26:00, yawano wrote: > > Just one ...
5 years, 10 months ago
(2015-02-19 11:09:54 UTC)
#5
On 2015/02/19 10:33:17, fukino wrote:
> On 2015/02/19 10:26:00, yawano wrote:
> > Just one question. Who uses expand_less.png?
>
> expand_less.png is not in use. Should I remove it?
> I thought, as we use expand_more, expand_more_active, and expand_less_active,
> absence of expand_less might be confusing. WDYT?
SGTM and lgtm. Thank you!
fukino
The CQ bit was checked by fukino@chromium.org
5 years, 10 months ago
(2015-02-19 11:10:35 UTC)
#6
Issue 939013002: Files.app: Update filetype icons.
(Closed)
Created 5 years, 10 months ago by fukino
Modified 5 years, 10 months ago
Reviewers: yawano
Base URL: https://chromium.googlesource.com/chromium/src.git@master
Comments: 0