Increase line-height to 38px and decrease column headers by 1 px in file manager

Increase line-height to 38px and decrease column headers by 1 px in file manager


BUG=chromium-os:25932, chromium-os:25934
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=124133

[bshe, 2012-02-28 01:38:17]

Hi Rob.

Could you please take a look at this minor UI tweaks? It follows kenmoore's
advice and these will likely not be final value.

Thanks,

[rginda, 2012-02-28 18:07:33]

lgtm

[commit-bot: I haz the power, 2012-02-28 18:31: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.

[rginda, 2012-02-28 18:40:54]

Dude, I'm totally a superstar.

[bshe, 2012-02-28 18:42:47]

On 2012/02/28 18:40:54, rginda wrote:
> Dude, I'm totally a superstar.

:) Thats what I think too.

[bshe, 2012-02-28 20:22:07]

+flackr 
"another" full super star committer

[flackr, 2012-02-28 20:27:02]

http://codereview.chromium.org/9481021/diff/1/chrome/browser/resources/file_m...
File chrome/browser/resources/file_manager/css/file_manager.css (right):

http://codereview.chromium.org/9481021/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/css/file_manager.css:696: height: 28px;
Doesn't this need to be updated as well?

[bshe, 2012-02-28 20:51:54]

Done. Thanks!

http://codereview.chromium.org/9481021/diff/1/chrome/browser/resources/file_m...
File chrome/browser/resources/file_manager/css/file_manager.css (right):

http://codereview.chromium.org/9481021/diff/1/chrome/browser/resources/file_m...
chrome/browser/resources/file_manager/css/file_manager.css:696: height: 28px;
On 2012/02/28 20:27:02, flackr wrote:
> Doesn't this need to be updated as well?

Done.

[flackr, 2012-02-28 20:55:50]

lgtm

[commit-bot: I haz the power, 2012-02-28 21:21:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bshe@chromium.org/9481021/8002

[commit-bot: I haz the power, 2012-02-28 22:45:24]

Try job failure for 9481021-8002 (retry) on linux_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2012-02-29 03:34:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bshe@chromium.org/9481021/8002

[commit-bot: I haz the power, 2012-02-29 06:03:17]

Change committed as 124133