Adjust the thumbnail view to match the specs.

Adjust the thumbnail view to match the specs.

This patch adjusts the grid view's styles to match the recent specs by removing checkbox and making the label transparent. Along the way, the redundant border (from webui) has been hidden.

TBR=yoshiki@chromium.org
TEST=Go to Files.app, enter a directory with images. Switch to the grid view.
BUG=239288
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=202802

[mtomasz, 2013-05-28 07:45:43]

@hirono: PTAL. Thanks.

[hirono, 2013-05-28 08:28:36]

https://codereview.chromium.org/15878007/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/css/file_manager.css (right):

https://codereview.chromium.org/15878007/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/css/file_manager.css:1210: margin-left:
8px;
"-webkit-margin-start: 8px;" ?

https://codereview.chromium.org/15878007/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/css/file_manager.css:1268: text-align:
left;
"text-align: start;"?

https://codereview.chromium.org/15878007/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/css/file_manager.css:1274: margin-left:
21px;
"-webkit-margin-start: 21px;" ?

[mtomasz, 2013-05-28 09:16:04]

https://codereview.chromium.org/15878007/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/css/file_manager.css (right):

https://codereview.chromium.org/15878007/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/css/file_manager.css:1210: margin-left:
8px;
On 2013/05/28 08:28:36, hirono wrote:
> "-webkit-margin-start: 8px;" ?

Done.

https://codereview.chromium.org/15878007/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/css/file_manager.css:1268: text-align:
left;
On 2013/05/28 08:28:36, hirono wrote:
> "text-align: start;"?

Done.

https://codereview.chromium.org/15878007/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/css/file_manager.css:1274: margin-left:
21px;
On 2013/05/28 08:28:36, hirono wrote:
> "-webkit-margin-start: 21px;" ?

Done.

[hirono, 2013-05-28 09:35:11]

https://codereview.chromium.org/15878007/diff/4001/chrome/browser/resources/f...
File chrome/browser/resources/file_manager/css/file_manager.css (right):

https://codereview.chromium.org/15878007/diff/4001/chrome/browser/resources/f...
chrome/browser/resources/file_manager/css/file_manager.css:1267: padding-left:
28px;
-webkit-padding-start ?

[mtomasz, 2013-05-28 09:48:27]

https://codereview.chromium.org/15878007/diff/4001/chrome/browser/resources/f...
File chrome/browser/resources/file_manager/css/file_manager.css (right):

https://codereview.chromium.org/15878007/diff/4001/chrome/browser/resources/f...
chrome/browser/resources/file_manager/css/file_manager.css:1267: padding-left:
28px;
On 2013/05/28 09:35:11, hirono wrote:
> -webkit-padding-start ?

Done.

[hirono, 2013-05-28 09:59:46]

On 2013/05/28 09:48:27, mtomasz wrote:
>
https://codereview.chromium.org/15878007/diff/4001/chrome/browser/resources/f...
> File chrome/browser/resources/file_manager/css/file_manager.css (right):
> 
>
https://codereview.chromium.org/15878007/diff/4001/chrome/browser/resources/f...
> chrome/browser/resources/file_manager/css/file_manager.css:1267: padding-left:
> 28px;
> On 2013/05/28 09:35:11, hirono wrote:
> > -webkit-padding-start ?
> 
> Done.

Thanks! lgtm!

[commit-bot: I haz the power, 2013-05-28 10:08:44]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[commit-bot: I haz the power, 2013-05-29 02:54:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/15878007/11001

[commit-bot: I haz the power, 2013-05-29 07:45:00]

Change committed as 202802