Provide feedback when list item are activated.

Provide feedback when list item are activated.

This CL provides touch-operation users with feedback earlier.
Menu items will be colored blue on touchstart.
Scrollable items (in directory tree and file list) will be colored gray on touchstart.
We don't use blue for them because touching them doesn't mean selecting them. (Users may have started scrolling)

BUG=404540
TEST=manually

Committed: https://crrev.com/5e178f4ddebd8cb018d8e4cf64dbb8132690add9
Cr-Commit-Position: refs/heads/master@{#293702}

[fukino, 2014-09-04 11:35:55]

fukino@chromium.org changed reviewers:
+ hirono@chromium.org

[fukino, 2014-09-04 11:36:43]

@hirono, PTAL the CL? Thanks!

[hirono, 2014-09-05 06:30:35]

Sorry for late.

https://codereview.chromium.org/540873002/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/foreground/css/file_manager.css (right):

https://codereview.chromium.org/540873002/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/foreground/css/file_manager.css:210:
background-color: rgb(225, 225, 225);
How about this one?

[fukino, 2014-09-05 06:43:45]

https://codereview.chromium.org/540873002/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/foreground/css/file_manager.css (right):

https://codereview.chromium.org/540873002/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/foreground/css/file_manager.css:210:
background-color: rgb(225, 225, 225);
On 2014/09/05 06:30:35, hirono wrote:
> How about this one?

Good catch!
#navigation-list doesn't exist anymore, so I removed styles for
#navigation-list.

[fukino, 2014-09-08 09:12:48]

On 2014/09/05 06:43:45, fukino wrote:
>
https://codereview.chromium.org/540873002/diff/1/ui/file_manager/file_manager...
> File ui/file_manager/file_manager/foreground/css/file_manager.css (right):
> 
>
https://codereview.chromium.org/540873002/diff/1/ui/file_manager/file_manager...
> ui/file_manager/file_manager/foreground/css/file_manager.css:210:
> background-color: rgb(225, 225, 225);
> On 2014/09/05 06:30:35, hirono wrote:
> > How about this one?
> 
> Good catch!
> #navigation-list doesn't exist anymore, so I removed styles for
> #navigation-list.

ping?

[hirono, 2014-09-08 09:20:53]

On 2014/09/08 09:12:48, fukino wrote:
> On 2014/09/05 06:43:45, fukino wrote:
> >
>
https://codereview.chromium.org/540873002/diff/1/ui/file_manager/file_manager...
> > File ui/file_manager/file_manager/foreground/css/file_manager.css (right):
> > 
> >
>
https://codereview.chromium.org/540873002/diff/1/ui/file_manager/file_manager...
> > ui/file_manager/file_manager/foreground/css/file_manager.css:210:
> > background-color: rgb(225, 225, 225);
> > On 2014/09/05 06:30:35, hirono wrote:
> > > How about this one?
> > 
> > Good catch!
> > #navigation-list doesn't exist anymore, so I removed styles for
> > #navigation-list.
> 
> ping?

lgtm!

[fukino, 2014-09-08 09:21:16]

On 2014/09/08 09:20:53, hirono wrote:
> On 2014/09/08 09:12:48, fukino wrote:
> > On 2014/09/05 06:43:45, fukino wrote:
> > >
> >
>
https://codereview.chromium.org/540873002/diff/1/ui/file_manager/file_manager...
> > > File ui/file_manager/file_manager/foreground/css/file_manager.css (right):
> > > 
> > >
> >
>
https://codereview.chromium.org/540873002/diff/1/ui/file_manager/file_manager...
> > > ui/file_manager/file_manager/foreground/css/file_manager.css:210:
> > > background-color: rgb(225, 225, 225);
> > > On 2014/09/05 06:30:35, hirono wrote:
> > > > How about this one?
> > > 
> > > Good catch!
> > > #navigation-list doesn't exist anymore, so I removed styles for
> > > #navigation-list.
> > 
> > ping?
> 
> lgtm!

Thanks!

[fukino, 2014-09-08 09:21:57]

The CQ bit was checked by fukino@chromium.org

[commit-bot: I haz the power, 2014-09-08 09:23:31]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fukino@chromium.org/540873002/40001

[commit-bot: I haz the power, 2014-09-08 10:21:37]

Committed patchset #3 (id:40001) as 8b85f0597c3910ebb9165f632a5a23e49245472a

[commit-bot: I haz the power, 2014-09-10 03:45:35]

Patchset 3 (id:??) landed as
https://crrev.com/5e178f4ddebd8cb018d8e4cf64dbb8132690add9
Cr-Commit-Position: refs/heads/master@{#293702}