[MD settings] redesign of side nav

chrome/browser/resources/settings/settings_menu/settings_menu.html

 <link rel="import" href="chrome://resources/cr_elements/icons.html">
 <link rel="import" href="chrome://resources/html/polymer.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-menu/paper-menu.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-menu/paper-submenu.html">
 <link rel="import" href="/icons.html">
 <link rel="import" href="/settings_shared_css.html">
 
 <dom-module id="settings-menu">
   <template>
@@ skipping 18 matching lines @@
         };
         --paper-menu-focused-item-after: {
           background: none;
         };
         background: none;  /* Overrides <paper-menu> default. */
         color: var(--settings-nav-grey);
         font-size: 13px;
         padding: 0;
       }
 
-      paper-submenu > div {
+      #basicPage {
         margin-top: 8px;
       }
 
-      paper-submenu {
+      .page-menu > div {
+        margin-top: 8px;
+      }
+
+      .page-menu {
         --paper-menu-selected-item: {
           font-weight: 500;
         };
       }
 
-      paper-submenu paper-menu {
+      .page-menu paper-menu {
         --paper-menu-selected-item: {
           color: var(--google-blue-500);
           font-weight: 500;
         };
       }
 
-      paper-submenu div {
+      .page-menu div {
         @apply(--layout-center);
         -webkit-margin-start: 24px;
         display: flex;
         font-size: 13px;
         font-weight: 500;
         min-height: 40px;
       }
 
       .separator {
         /* Per Alan@, this line is different from the other separator lines. */
         border-bottom: 1px solid rgba(0, 0, 0, 0.08);
         margin-top: 8px;
       }
     </style>
     <paper-menu name="root-menu">
-      <paper-submenu data-page="basic" id="basicPage" opened="{{basicOpened_}}">
-        <div class="menu-trigger" data-section="" on-tap="openPage_">
-          <span>$i18n{basicPageTitle}</span>
-          <iron-icon icon="[[arrowState_(basicOpened_)]]"></iron-icon>
-        </div>
+      <div class="page-menu" data-page="basic" id="basicPage"
+          hidden$="!basicOpened_">

[Dan Beam, 2016/05/31 23:28:24]

where is basicOpened_ defined?  also, why would we ever hide these?

[dschuyler, 2016/06/02 01:05:37]

Done.

         <paper-menu attr-for-selected="data-section" class="menu-content"
             data-page="basic" id="basicMenu">
 <if expr="chromeos">
           <div data-section="internet" on-tap="openPage_">
             <iron-icon icon="settings:network-wifi"></iron-icon>
             $i18n{internetPageTitle}
           </div>
 </if>
           <div data-section="people" on-tap="openPage_">
             <iron-icon icon="settings:people"></iron-icon>
@@ skipping 17 matching lines @@
           <div data-section="defaultBrowser" on-tap="openPage_">
             <iron-icon icon="settings:web"></iron-icon>
             $i18n{defaultBrowser}
           </div>
 </if>
           <div data-section="onStartup" on-tap="openPage_">
             <iron-icon icon="settings:power-settings-new"></iron-icon>
             $i18n{onStartup}
           </div>
         </paper-menu>
-      </paper-submenu>
-      <paper-submenu id="advancedPage" data-page="advanced"
+      </div>
+      <paper-submenu class="page-menu" id="advancedPage" data-page="advanced"
           opened="{{advancedOpened_}}">
-        <div class="menu-trigger" data-section="" on-tap="openPage_">
+        <div class="menu-trigger" data-section="" on-tap="toggleAdvancedPage_">
           <span>$i18n{advancedPageTitle}</span>
           <iron-icon icon="[[arrowState_(advancedOpened_)]]"></iron-icon>
         </div>
         <paper-menu attr-for-selected="data-section" class="menu-content"
             data-page="advanced" id="advancedMenu">
 <if expr="chromeos">
           <div data-section="dateTime" on-tap="openPage_">
             <iron-icon icon="settings:access-time"></iron-icon>
             $i18n{dateTimePageTitle}
           </div>
@@ skipping 32 matching lines @@
             $i18n{systemPageTitle}
           </div>
 </if>
           <div data-section="reset" on-tap="openPage_">
             <iron-icon icon="settings:restore"></iron-icon>
             $i18n{resetPageTitle}
           </div>
         </paper-menu>
       </paper-submenu>
       <div class="separator"></div>
-      <paper-submenu data-page="about">
+      <paper-submenu class="page-menu" data-page="about">
         <div class="menu-trigger" on-tap="openPage_" data-section="">
           $i18n{aboutPageTitle}
         </div>
       </paper-submenu>
     </paper-menu>
   </template>
   <script src="settings_menu.js"></script>
 </dom-module>