MD Settings: Fix internet focus and styling

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

 <link rel="import" href="chrome://resources/cr_elements/cr_expand_button/cr_expand_button.html">
 <link rel="import" href="chrome://resources/cr_elements/icons.html">
 <link rel="import" href="chrome://resources/cr_elements/network/cr_network_list.html">
 <link rel="import" href="chrome://resources/cr_elements/network/cr_network_list_item.html">
 <link rel="import" href="chrome://resources/html/polymer.html">
 <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">
 <link rel="import" href="network_siminfo.html">
 
 <dom-module name="network-summary-item">
   <template>
     <style include="settings-shared">
       paper-spinner {
         height: 20px;
@@ skipping 27 matching lines @@
 
       #deviceEnabledButton {
         -webkit-margin-start: 10px;
       }
 
       #networkList {
         margin: 5px 0 10px;
         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_)]]"
             hidden$="[[!scanningIsVisible_(deviceState)]]">
         </paper-spinner>
       </div>
 
       <template is="dom-if" if="[[showSimInfo_(deviceState)]]">
         <network-siminfo editable class="flex"
             network-properties="[[getCellularState_(deviceState)]]"
             networking-private="[[networkingPrivate]]">
         </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 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>
-      </div>
+      </template>
+
     </div>
     <template is="dom-if"
         if="[[networksDomIfIsTrue_(expanded_, wasExpanded_)]]">
       <iron-collapse
           opened="[[networksIronCollapseIsOpened_(expanded_, wasExpanded_)]]">
         <div class="list-frame">
           <cr-network-list id="networkList" max-height="[[maxHeight]]"
               networks="[[networkStateList]]" show-buttons>
           </cr-network-list>
           <template is="dom-if"
               if="[[knownNetworksIsVisible_(activeNetworkState)]]">
             <div class="button-row" actionable on-tap="onKnownNetworksTap_">
               <a is="action-link">$i18n{knownNetworksButton}</a>
             </div>
           </template>
         </div>
       </iron-collapse>
     </template>
   </template>
   <script src="network_summary_item.js"></script>
 </dom-module>