User Manager UI fixes

User Manager UI fixes

- use high dpi resources
- add a logo to the tray
- css fixes: increase the tray height, hover/focused states of pods,
increase the profile name box
- ensure non-focused pods are faded out
- update/make equal the padding between pod rows and columns
- change the style of the action box drop down arrow

See crbug for mocks.
Screenshots:
https://drive.google.com/file/d/0B1B1Up4p2NRMUDFqWmgtSGVlbmM/edit?usp=sharing

BUG=380273
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=277815

[Nikita (slow), 2014-06-16 15:28:10]

lgtm

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/log...
File chrome/browser/resources/login/user_pod_row.css (left):

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/log...
chrome/browser/resources/login/user_pod_row.css:255: border: 0 !important;
I thought that this change may break something in CrOS UI but it appears to have
no effect on us.

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/log...
File chrome/browser/resources/login/user_pod_row.js (right):

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/log...
chrome/browser/resources/login/user_pod_row.js:1911: pod.top = row * (height +
rowPadding);
Should be
pod.top = rowPadding + row * height

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/use...
File chrome/browser/resources/user_manager/control_bar.css (right):

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/use...
chrome/browser/resources/user_manager/control_bar.css:47: border-right: 1px
solid #e2e2e2;
border-right / margin-right: does this mean that rtl case is not properly
covered?

[noms (inactive), 2014-06-16 18:10:56]

I've made a few changes. Please take another look. :)

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/log...
File chrome/browser/resources/login/user_pod_row.css (left):

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/log...
chrome/browser/resources/login/user_pod_row.css:255: border: 0 !important;
Yeah, I don't know why it was there (I couldn't find a border it would have had
conflicts with...)
On 2014/06/16 15:28:10, Nikita Kostylev wrote:
> I thought that this change may break something in CrOS UI but it appears to
have
> no effect on us.

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/log...
File chrome/browser/resources/login/user_pod_row.js (right):

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/log...
chrome/browser/resources/login/user_pod_row.js:1911: pod.top = row * (height +
rowPadding);
So I think on desktop we want the spacing to always be equal, regardless of how
many rows there are, which means the padding needs to be added to each row. I
think on CrOS the intention is to adjust the padding based on how many rows
there are. I've added an if statement to pick the right approach based on the
platform.

On 2014/06/16 15:28:10, Nikita Kostylev wrote:
> Should be
> pod.top = rowPadding + row * height

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/use...
File chrome/browser/resources/user_manager/control_bar.css (right):

https://codereview.chromium.org/330283004/diff/1/chrome/browser/resources/use...
chrome/browser/resources/user_manager/control_bar.css:47: border-right: 1px
solid #e2e2e2;
On 2014/06/16 15:28:10, Nikita Kostylev wrote:
> border-right / margin-right: does this mean that rtl case is not properly
> covered?

Done.

[Nikita (slow), 2014-06-17 09:39:23]

lgtm

https://codereview.chromium.org/330283004/diff/80001/chrome/browser/resources...
File chrome/browser/resources/login/user_pod_row.js (right):

https://codereview.chromium.org/330283004/diff/80001/chrome/browser/resources...
chrome/browser/resources/login/user_pod_row.js:1911: // On desktop, we want the
rows to always be equally spaced.
nit: Insert empty line before comment.

[noms (inactive), 2014-06-17 13:55:44]

https://codereview.chromium.org/330283004/diff/80001/chrome/browser/resources...
File chrome/browser/resources/login/user_pod_row.js (right):

https://codereview.chromium.org/330283004/diff/80001/chrome/browser/resources...
chrome/browser/resources/login/user_pod_row.js:1911: // On desktop, we want the
rows to always be equally spaced.
On 2014/06/17 09:39:23, Nikita Kostylev wrote:
> nit: Insert empty line before comment.

Done.

[noms (inactive), 2014-06-17 13:55:56]

The CQ bit was unchecked by noms@chromium.org

[noms (inactive), 2014-06-17 13:55:56]

The CQ bit was checked by noms@chromium.org

[noms (inactive), 2014-06-17 13:55:57]

The CQ bit was checked by noms@chromium.org

[commit-bot: I haz the power, 2014-06-17 13:57:10]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/noms@chromium.org/330283004/100001

[commit-bot: I haz the power, 2014-06-17 18:39:11]

Change committed as 277815