Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(75)

Unified Diff: chrome/browser/resources/md_history/history_card.html

Issue 1643693003: MD History: Implement search functionality. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@patch_to_be_uploaded
Patch Set: Address reviewer comments. Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/md_history/history_card.html
diff --git a/chrome/browser/resources/md_history/history_card.html b/chrome/browser/resources/md_history/history_card.html
index 7c0c8a0b4c2c2b245eede5d31ad09175484e7f15..b083bcca18a5748d6d5b7b0c563089f18db7af1b 100644
--- a/chrome/browser/resources/md_history/history_card.html
+++ b/chrome/browser/resources/md_history/history_card.html
@@ -12,6 +12,10 @@
padding: 0 24px 20px;
}
+ :host([hidden]) {
+ display: none !important;
+ }
+
#main-container {
@apply(--shadow-elevation-2dp);
background: #fff;
@@ -46,19 +50,22 @@
</style>
<div id="main-container">
- <div id="date-accessed">[[historyDate]]</div>
+ <div id="date-accessed">
+ {{cardTitle_(historyItems.length, searchTerm, historyTitle)}}
+ </div>
<div id="history-item-list">
<template is="dom-repeat" items="{{historyItems}}"
as="historyItem">
- <history-item time-accessed_="[[historyItem.dateTimeOfDay]]"
+ <history-item time-accessed_="[[historyItem.visibleTimestamp]]"
website-title_="[[historyItem.title]]"
website-domain_="[[historyItem.domain]]"
website-url_="[[historyItem.url]]"
starred="[[historyItem.starred]]"
selected="{{historyItem.selected}}"
- timestamp_="[[historyItem.time]]">
+ timestamp_="[[historyItem.time]]"
+ search-term="[[searchTerm]]">
</history-item>
- <template is="dom-if" if="[[needsTimeGap_(index, historyItem)]]">
+ <template is="dom-if" if="[[needsTimeGap_(index, historyItem, searchTerm)]]">
tsergeant 2016/02/08 22:59:01 Nit: Wrap to 80 characters
hsampson 2016/02/09 02:17:51 Done.
<div id="time-gap-separator"></div>
</template>
</template>

Powered by Google App Engine
This is Rietveld 408576698