Fix Print Preview navbar links layout

Fix Print Preview navbar links layout (broken by crrev.com/668983004).

BUG=399376, 427388
Committed: https://crrev.com/e60561ddb24168c6682466b91567b6f9d54cae94
Cr-Commit-Position: refs/heads/master@{#301915}

[Aleksey Shlyapnikov, 2014-10-28 19:04:11]

alekseys@chromium.org changed reviewers:
+ dbeam@chromium.org

[Dan Beam, 2014-10-28 20:38:38]

lgtm

https://codereview.chromium.org/680403002/diff/1/chrome/browser/resources/pri...
File chrome/browser/resources/print_preview/print_preview.css (right):

https://codereview.chromium.org/680403002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/print_preview.css:128: text-align: start;
^ what's this for?

https://codereview.chromium.org/680403002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/print_preview.css:129: text-decoration:
none;
^ don't think you need the text-decoration related styles

[Aleksey Shlyapnikov, 2014-10-28 23:24:06]

https://codereview.chromium.org/680403002/diff/1/chrome/browser/resources/pri...
File chrome/browser/resources/print_preview/print_preview.css (right):

https://codereview.chromium.org/680403002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/print_preview.css:128: text-align: start;
On 2014/10/28 20:38:38, Dan Beam wrote:
> ^ what's this for?

Not sure why it was added, but we don't need it now for sure.

https://codereview.chromium.org/680403002/diff/1/chrome/browser/resources/pri...
chrome/browser/resources/print_preview/print_preview.css:129: text-decoration:
none;
On 2014/10/28 20:38:38, Dan Beam wrote:
> ^ don't think you need the text-decoration related styles

Done.

[Aleksey Shlyapnikov, 2014-10-29 00:37:58]

The CQ bit was checked by alekseys@chromium.org

[commit-bot: I haz the power, 2014-10-29 00:39:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/680403002/20001

[Dan Beam, 2014-10-29 00:41:49]

still lgtm

https://codereview.chromium.org/680403002/diff/20001/chrome/browser/resources...
File chrome/browser/resources/print_preview/print_preview.css (right):

https://codereview.chromium.org/680403002/diff/20001/chrome/browser/resources...
chrome/browser/resources/print_preview/print_preview.css:131: text-decoration:
underline;
is this necessary as well?

https://codereview.chromium.org/680403002/diff/20001/chrome/browser/resources...
chrome/browser/resources/print_preview/print_preview.css:135: color: rgba(0, 0,
0, .5);
why is this here?  specificity issues?  intentional differences from the rest of
chrome?  in general it'd be best to just have everybody agree.

[Aleksey Shlyapnikov, 2014-10-29 01:25:10]

The CQ bit was unchecked by alekseys@chromium.org

[Aleksey Shlyapnikov, 2014-10-29 02:15:21]

https://codereview.chromium.org/680403002/diff/20001/chrome/browser/resources...
File chrome/browser/resources/print_preview/print_preview.css (right):

https://codereview.chromium.org/680403002/diff/20001/chrome/browser/resources...
chrome/browser/resources/print_preview/print_preview.css:131: text-decoration:
underline;
On 2014/10/29 00:41:48, Dan Beam wrote:
> is this necessary as well?

Done.

https://codereview.chromium.org/680403002/diff/20001/chrome/browser/resources...
chrome/browser/resources/print_preview/print_preview.css:135: color: rgba(0, 0,
0, .5);
On 2014/10/29 00:41:48, Dan Beam wrote:
> why is this here?  specificity issues?  intentional differences from the rest
of
> chrome?  in general it'd be best to just have everybody agree.

I suppose, links were made gray to match the rest of the Print Preview UI in the
disabled state.

Thank you for asking, I realized that links cannot be disabled the same way as
buttons. Fixed that too.

[Aleksey Shlyapnikov, 2014-10-29 19:20:14]

The CQ bit was checked by alekseys@chromium.org

[commit-bot: I haz the power, 2014-10-29 19:21:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/680403002/60001

[commit-bot: I haz the power, 2014-10-29 20:28:03]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2014-10-29 20:28:51]

Patchset 4 (id:??) landed as
https://crrev.com/e60561ddb24168c6682466b91567b6f9d54cae94
Cr-Commit-Position: refs/heads/master@{#301915}