MD History: Refactored design for displaying history information

chrome/browser/resources/md_history/history_item.html

Index: chrome/browser/resources/md_history/history_item.html
diff --git a/chrome/browser/resources/md_history/history_item.html b/chrome/browser/resources/md_history/history_item.html
index 7959d95685fb8dd916d3ce497e95d75c27f79a4e..d348fc29c46fdd61952c7573efffbe4f20efde52 100644
--- a/chrome/browser/resources/md_history/history_item.html
+++ b/chrome/browser/resources/md_history/history_item.html
@@ -1,20 +1,64 @@
 <link rel="import" href="chrome://resources/polymer/v1_0/polymer/polymer.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/iron-flex-layout.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icon/iron-icon.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-icons/iron-icons.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-button.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-checkbox/paper-checkbox.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-styles/shadow.html">
 <link rel="import" href="chrome://resources/html/util.html">
 
 <dom-module id="history-item">
   <template>
     <style>
+      :host {
+        @apply(--layout-center);
+        @apply(--layout-vertical);
+        padding: 0 24px;
+      }
+
       #main-container {
+        background: #fff;
+        max-width: 960px;
+        min-width: 500px;
+        width: 100%;
+      }
+
+      :host([is-card-start]) #main-container {
+        margin-top: 20px;
+      }
+
+      #date-accessed {
+        display: none;
+      }
+
+      :host([is-card-start]) #date-accessed {
+        @apply(--layout-center);
+        @apply(--layout-horizontal);
+        -webkit-padding-start: 20px;
+        background: #fafafa;
+        border-bottom: 1px solid rgba(0, 0, 0, 0.14);
+        border-radius: 2px 2px 0 0;
+        color: #333;
+        font-size: 14px;
+        font-weight: 500;
+        height: 48px;
+      }
+
+      #item-container {
         @apply(--layout-center);
         @apply(--layout-horizontal);
         min-height: 40px;
       }
 
+      :host([is-card-start]) #item-container {
+        padding-top: 8px;
+      }
+
+      :host([is-card-end]) #item-container {
+        padding-bottom: 8px;
+      }
+
       #titleAndDomain {
         @apply(--layout-center);
         @apply(--layout-flex);
@@ -84,24 +128,33 @@
         visibility: hidden;
       }
 
-      :host([starred]) #bookmark {
-        visibility: visible;
+      #time-gap-separator {
+        -webkit-border-start: 1px solid #888;
+        -webkit-margin-start: 77px;
+        height: 15px;
       }
     </style>
+
     <div id="main-container">
-      <paper-checkbox id="checkbox" on-tap="onCheckboxSelected_"
-          checked="{{selected}}">
-      </paper-checkbox>
-      <span id="time">{{timeAccessed_}}</span>
-      <div id="website-icon"></div>
-      <div id="titleAndDomain">
-        <a href="{{websiteUrl_}}" id="title">{{websiteTitle_}}</a>
-        <span id="domain">{{websiteDomain_}}</span>
+      <div id="date-accessed">[[historyDate]]</div>
+      <div id="item-container">
+        <paper-checkbox id="checkbox" on-tap="onCheckboxSelected_"
+            checked="{{selected}}">
+        </paper-checkbox>
+        <span id="time">{{timeAccessed}}</span>
+        <div id="website-icon"></div>
+        <div id="titleAndDomain">
+          <a href="{{websiteUrl}}" id="title">{{websiteTitle}}</a>

[tsergeant, 2016/02/08 02:23:59]

While you're here,

when you use polymer to bind to a native attribute, you're supposed to use $=

So this should be <a href$="{{websiteUrl}}" ...>

[yingran, 2016/02/08 03:20:36]

Done.

+          <span id="domain">{{websiteDomain}}</span>
+        </div>
+        <iron-icon icon="star" id="bookmark"></iron-icon>
+        <paper-icon-button icon="more-vert" id="menu-button"
+            on-tap="onMenuButtonTap_">
+        </paper-icon-button>
       </div>
-      <iron-icon icon="star" id="bookmark"></iron-icon>
-      <paper-icon-button icon="more-vert" id="menu-button"
-          on-tap="onMenuButtonTap_">
-      </paper-icon-button>
+      <template is="dom-if" if="{{hasTimeGap}}">
+        <div id="time-gap-separator"></div>
+      </template>
     </div>
   </template>
   <script src="chrome://history/history_item.js"></script>