MD Settings: polish up some top-level buttons

MD Settings: polish up some top-level buttons

This removes text-transform none that undoes capitalization.
Yes, this does expose some oddities for rows that are implemented as

<paper-button>
  <div>
  <div class="secondary>

I think we should implement those instead with:

<paper-ripple>
  <div>
  <div class="secondary">

But it depends on the desired focus behavior.

In general, we should be using <paper-button> when the mocks are
ALL UPPER, and <a href> for lowercase links and <a is="action-link">
for lowercased links that dont actually go anywhere
(i.e. "Add a search engine")

R=dschuyler@chromium.org
BUG=425627
Committed: https://crrev.com/28167989a78878b282fdd4a882ec8a5c16ced975
Cr-Commit-Position: refs/heads/master@{#377153}

[Dan Beam, 2016-02-20 00:12:53]

Patchset #1 (id:1) has been deleted

[Dan Beam, 2016-02-20 00:13:59]

Description was changed from

==========
MD Settings: polish up some top-level buttons

This removes text-transform none that undoes capitalization

In general, we should be using <paper-button> when the mocks are
ALL UPPER, and <a href> for lowercase links and <a is="action-link">
for lowercased links that dont actually go anywhere
(i.e. "Add a search engine")

R=dschuyler@chromium.org
BUG=425627
==========

to

==========
MD Settings: polish up some top-level buttons

This removes text-transform none that undoes capitalization.
Yes, this does expose some oddities for rows that are implemented as

<paper-button>
  <div>
  <div class="secondary>

I think we should implement those instead with:

<paper-ripple>
  <div>
  <div class="secondary">

But it depends on the desired focus behavior.

In general, we should be using <paper-button> when the mocks are
ALL UPPER, and <a href> for lowercase links and <a is="action-link">
for lowercased links that dont actually go anywhere
(i.e. "Add a search engine")

R=dschuyler@chromium.org
BUG=425627
==========

[Dan Beam, 2016-02-20 00:14:12]

http://imgur.com/a/6w8pM

[Dan Beam, 2016-02-23 02:51:48]

ping

[dschuyler, 2016-02-23 21:24:22]

lgtm

[Dan Beam, 2016-02-23 22:21:30]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2016-02-23 22:21:30]

The patchset sent to the CQ was uploaded after l-g-t-m from
dschuyler@chromium.org
Link to the patchset: https://codereview.chromium.org/1709413003/#ps80001
(title: "id")

[commit-bot: I haz the power, 2016-02-23 22:22:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1709413003/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1709413003/80001

[commit-bot: I haz the power, 2016-02-23 22:41:43]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-23 22:41:44]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  android_clang_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_clobber_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)
  win_chromium_compile_dbg_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Dan Beam, 2016-02-23 23:59:43]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2016-02-23 23:59:44]

The patchset sent to the CQ was uploaded after l-g-t-m from
dschuyler@chromium.org
Link to the patchset: https://codereview.chromium.org/1709413003/#ps100001
(title: "merge")

[commit-bot: I haz the power, 2016-02-24 00:05:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1709413003/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1709413003/100001

[commit-bot: I haz the power, 2016-02-24 00:44:29]

Description was changed from

==========
MD Settings: polish up some top-level buttons

This removes text-transform none that undoes capitalization.
Yes, this does expose some oddities for rows that are implemented as

<paper-button>
  <div>
  <div class="secondary>

I think we should implement those instead with:

<paper-ripple>
  <div>
  <div class="secondary">

But it depends on the desired focus behavior.

In general, we should be using <paper-button> when the mocks are
ALL UPPER, and <a href> for lowercase links and <a is="action-link">
for lowercased links that dont actually go anywhere
(i.e. "Add a search engine")

R=dschuyler@chromium.org
BUG=425627
==========

to

==========
MD Settings: polish up some top-level buttons

This removes text-transform none that undoes capitalization.
Yes, this does expose some oddities for rows that are implemented as

<paper-button>
  <div>
  <div class="secondary>

I think we should implement those instead with:

<paper-ripple>
  <div>
  <div class="secondary">

But it depends on the desired focus behavior.

In general, we should be using <paper-button> when the mocks are
ALL UPPER, and <a href> for lowercase links and <a is="action-link">
for lowercased links that dont actually go anywhere
(i.e. "Add a search engine")

R=dschuyler@chromium.org
BUG=425627
==========

[commit-bot: I haz the power, 2016-02-24 00:44:30]

Committed patchset #5 (id:100001)

[commit-bot: I haz the power, 2016-02-24 00:46:00]

Description was changed from

==========
MD Settings: polish up some top-level buttons

This removes text-transform none that undoes capitalization.
Yes, this does expose some oddities for rows that are implemented as

<paper-button>
  <div>
  <div class="secondary>

I think we should implement those instead with:

<paper-ripple>
  <div>
  <div class="secondary">

But it depends on the desired focus behavior.

In general, we should be using <paper-button> when the mocks are
ALL UPPER, and <a href> for lowercase links and <a is="action-link">
for lowercased links that dont actually go anywhere
(i.e. "Add a search engine")

R=dschuyler@chromium.org
BUG=425627
==========

to

==========
MD Settings: polish up some top-level buttons

This removes text-transform none that undoes capitalization.
Yes, this does expose some oddities for rows that are implemented as

<paper-button>
  <div>
  <div class="secondary>

I think we should implement those instead with:

<paper-ripple>
  <div>
  <div class="secondary">

But it depends on the desired focus behavior.

In general, we should be using <paper-button> when the mocks are
ALL UPPER, and <a href> for lowercase links and <a is="action-link">
for lowercased links that dont actually go anywhere
(i.e. "Add a search engine")

R=dschuyler@chromium.org
BUG=425627

Committed: https://crrev.com/28167989a78878b282fdd4a882ec8a5c16ced975
Cr-Commit-Position: refs/heads/master@{#377153}
==========

[commit-bot: I haz the power, 2016-02-24 00:46:00]

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