MD History: Fix overlapping of sidebar footer in very short windows

chrome/browser/resources/md_history/side_bar.html

Index: chrome/browser/resources/md_history/side_bar.html
diff --git a/chrome/browser/resources/md_history/side_bar.html b/chrome/browser/resources/md_history/side_bar.html
index 7dbcc88b3998d10caf2df97a44f34a8e64bad489..9a36dbfda6e0ad31d39a522962979ea50e0e65b0 100644
--- a/chrome/browser/resources/md_history/side_bar.html
+++ b/chrome/browser/resources/md_history/side_bar.html
@@ -11,14 +11,21 @@
   <template>
     <style include="shared-style">
       :host {
-        display: block;
+        display: flex;
+        flex-direction: column;

[lshang, 2016/10/12 05:15:29]

nit: There is only one single iron-selector element inside of history-side-bar,
so this is probably not needed. The only difference I can see is the position of
iron-icon besides "Clearing browsing data" when the scroll bar shows.

[tsergeant, 2016/10/12 06:06:45]

Done.

         height: 100%;
-        padding-top: 5px;
+        overflow-x: hidden;
+        overflow-y: auto;
         width: var(--side-bar-width);
       }
 
+      :host([drawer]) {
+        height: calc(100% - var(--toolbar-height));

[lshang, 2016/10/12 05:15:29]

nit: could this be calc(100% - var(|height of drawer header|))? Even though
drawer-header's height equals var(--toolbar-height), it would be more clear that
the height of rest side bar part is calculated from the side bar header, within
the same element, not from reference of toolbar element, imo.

[tsergeant, 2016/10/12 06:06:45]

As discussed, I'm leaving this as-is without renaming.

+      }
+
       div.separator {
         background-color: rgba(0, 0, 0, 0.08);
+        flex-shrink: 0;
         height: 1px;
         margin: 8px 0;
       }
@@ -40,6 +47,10 @@
         -webkit-user-select: none;
         background-color: transparent;
         color: #5a5a5a;
+        display: flex;
+        flex: 1;
+        flex-direction: column;
+        padding-top: 5px;

[lshang, 2016/10/12 05:15:29]

nit: margins between "Tabs from other devices/Clear browsing data" and the
separator line are both 8px, so set padding-top: 8px here probably is better, to
be consistent.

Or you can set it to 10px, as what it is now on tot.

[tsergeant, 2016/10/12 06:06:45]

I've set it to 8px, which matches Settings

       }
 
       iron-selector > a {
@@ -59,22 +70,17 @@
 
       iron-selector > a.iron-selected {
         color: var(--google-blue-500);
-        font-weight: 500;
+      }
+
+      #spacer {
+        flex: 1;
       }
 
       #footer {
-        bottom: 0;
         color: var(--paper-grey-600);
-        position: absolute;
         width: var(--side-bar-width);
       }
 
-      :host([drawer]) #footer {
-        /* This compensates the 120px by which app-drawer protrudes under
-         * the viewport. */
-        bottom: 120px;
-      }
-
       #footer-text {
         -webkit-padding-end: 16px;
         -webkit-padding-start: 24px;
@@ -108,6 +114,7 @@
         <iron-icon icon="cr:open-in-new"></iron-icon>
         <paper-ripple id="cbd-ripple"></paper-ripple>
       </a>
+      <div id="spacer"></div>
       <div id="footer" hidden="[[!showFooter]]">
         <div class="separator"></div>
         <div id="footer-text">$i18nRaw{sidebarFooter}</div>