MD History: Refactored design for displaying history information

chrome/browser/resources/md_history/history_list.html

 <link rel="import" href="chrome://resources/polymer/v1_0/polymer/polymer.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/iron-list/iron-list.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-item/paper-item.html">
 <link rel="import" href="chrome://resources/polymer/v1_0/paper-menu/paper-menu.html">
 <link rel="import" href="chrome://resources/html/cr/ui/position_util.html">
-<link rel="import" href="chrome://history/history_card.html">
+<link rel="import" href="chrome://history/history_item.html">
 
-<dom-module id="history-card-manager">
+<dom-module id="history-list">
   <template>
     <style>
       :host {
         display: flex;
         flex-direction: column;
       }
 
-      :host([menu-open]) #overflow-menu {
-        display: block;
-      }
-
       #overflow-menu {
         @apply(--shadow-elevation-2dp);
         border-radius: 2px;
         display: none;
         left: 0;
         position: absolute;
         top: 0;
       }
 
+      :host([menu-open]) #overflow-menu {
+        display: block;
+      }
+

[calamity, 2016/02/09 12:25:29]

This block seems to have moved. May as well leave it where it was.

[yingran, 2016/02/11 00:41:42]

am confused

[tsergeant, 2016/02/11 02:43:02]

My $0.02: Leave this block where it is now. It modifies the behavior of the
#overflow-menu block, so it makes more sense for it to be after it rather than
before it.

[yingran, 2016/02/12 02:03:05]

Done.

       #infinite-list {
         flex: 1;
-        padding: 40px 0 0 0;
+        padding: 20px 0 0 0;
       }
 
       paper-item {
         -webkit-user-select: none;
         cursor: pointer;
         font: inherit;
       }
 
       paper-item:hover {
         background: #eaeaea;
       }
     </style>
-    <iron-list items="{{historyDataByDay_}}" as="historyDay" id="infinite-list"
+    <iron-list items="{{historyData}}" as="historyDay" id="infinite-list"
         on-scroll="scrollHandler_">
       <template>
-        <history-card history-date="{{historyDay.date}}"
-            history-items="{{historyDay.historyItems}}">
-        </history-card>
+        <history-item history-date="[[historyDay.dateRelativeDay]]"
+            website-title="[[historyDay.title]]"
+            website-url="[[historyDay.url]]"
+            website-domain="[[historyDay.domain]]"
+            time-accessed="[[historyDay.dateTimeOfDay]]"
+            starred="[[historyDay.starred]]"
+            timestamp="[[historyDay.time]]"
+            selected="{{historyDay.selected}}"
+            is-card-start="[[historyDay.isFirstItem]]"
+            is-card-end="[[historyDay.isLastItem]]"
+            has-time-gap="[[historyDay.needsTimeGap]]">
+        </history-item>
       </template>
     </iron-list>
     <paper-menu id="overflow-menu">
       <paper-item i18n-content="moreFromSite" on-tap="onMoreFromSiteTap_">
       </paper-item>
       <paper-item i18n-content="removeFromHistory"
           on-tap="onRemoveFromHistoryTap_">
       </paper-item>
     </paper-menu>
   </template>
-  <script src="chrome://history/history_card_manager.js"></script>
+  <script src="chrome://history/history_list.js"></script>
 </dom-module>