MD Settings: Fix internet focus and styling

chrome/browser/resources/settings/internet_page/network_summary_item.html

Index: chrome/browser/resources/settings/internet_page/network_summary_item.html
diff --git a/chrome/browser/resources/settings/internet_page/network_summary_item.html b/chrome/browser/resources/settings/internet_page/network_summary_item.html
index fa4de37a16be2abfdc61aa288302c1591028812c..1793d36b6aee0691997fcfcbfc2266cb807ef3b6 100644
--- a/chrome/browser/resources/settings/internet_page/network_summary_item.html
+++ b/chrome/browser/resources/settings/internet_page/network_summary_item.html
@@ -6,6 +6,7 @@
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-a11y-keys-behavior/iron-a11y-keys-behavior.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-collapse/iron-collapse.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper-icon-button-light.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-spinner/paper-spinner.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-toggle-button/paper-toggle-button.html">
 <link rel="import" href="/settings_shared_css.html">
@@ -53,10 +54,9 @@
         max-height: 400px;
       }
     </style>
-    <div class="settings-box two-line">
-      <div id="details" selectable
-          no-flex$="[[showSimInfo_(deviceState)]]"
-          on-tap="onDetailsTap_" tabindex$="[[getTabIndex_(deviceState)]]">
+    <div class="settings-box two-line" actionable on-tap="onDetailsTap_">

[dschuyler, 2016/12/09 01:15:52]

optional:
I don't think there's a style guide on it, but I prefer that the attributes are
in alphabetical order.

[stevenjb, 2016/12/09 01:29:30]

Yeah, there's not. I tend to prefer: id class attributes-in-alpha-order events.

I can change if there is a different consensus.

+      <div id="details"
+          no-flex$="[[showSimInfo_(deviceState)]]">
         <cr-network-list-item item="[[activeNetworkState]]" class="flex">
         </cr-network-list-item>
         <paper-spinner active="[[scanningIsActive_(deviceState, expanded_)]]"
@@ -71,26 +71,29 @@
         </network-siminfo>
       </template>
 
-      <div>
-        <div id="buttons" class="layout horizontal">
-          <template is="dom-if"
-              if="[[showDetailsIsVisible_(deviceState, networkStateList)]]">
-            <paper-icon-button icon="cr:settings" on-tap="onShowDetailsTap_">
-            </paper-icon-button>
-          </template>
-          <template is="dom-if"
-              if="[[expandIsVisible_(deviceState, networkStateList)]]">
-            <cr-expand-button id="expandListButton" expanded="{{expanded_}}">
-            </cr-expand-button>
-          </template>
-          <template is="dom-if" if="[[enableIsVisible_(deviceState)]]">
-            <paper-toggle-button  id="deviceEnabledButton"
-                checked="[[deviceIsEnabled_(deviceState)]]"
-                on-tap="onDeviceEnabledTap_">
-            </paper-toggle-button>
-          </template>
-        </div>
+      <div id="buttons" class="layout horizontal">
+        <template is="dom-if"
+            if="[[expandIsVisible_(deviceState, networkStateList)]]">
+          <cr-expand-button id="expandListButton" expanded="{{expanded_}}">
+          </cr-expand-button>
+        </template>
+        <template is="dom-if"
+            if="[[!expandIsVisible_(deviceState, networkStateList)]]">
+          <button class="subpage-arrow" is="paper-icon-button-light"
+              on-tap="onShowDetailsTap_">
+          </button>
+        </template>
       </div>
+
+      <template is="dom-if" if="[[enableIsVisible_(deviceState)]]">
+        <div class="secondary-action">
+          <paper-toggle-button  id="deviceEnabledButton"
+              checked="[[deviceIsEnabled_(deviceState)]]"
+              on-tap="onDeviceEnabledTap_">
+          </paper-toggle-button>
+        </div>
+      </template>
+
     </div>
     <template is="dom-if"
         if="[[networksDomIfIsTrue_(expanded_, wasExpanded_)]]">