Chromium Code Reviews| 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 18b9cf8eb88094a121610122aa7f7828fea4aece..c7c84ec76c97c3267d333b7c295b0ea1ceb66dd7 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 24px; |
| } |
| + :host([hidden]) { |
|
tsergeant
2016/02/02 02:57:36
Is this needed? I can't see you setting hidden any
hsampson
2016/02/03 02:37:58
Yes this is needed so that the iron-list can hide
|
| + display: none !important; |
| + } |
| + |
| #main-container { |
| @apply(--shadow-elevation-2dp); |
| background: #fff; |
| @@ -45,7 +49,14 @@ |
| </style> |
| <div id="main-container"> |
| - <div id="date-accessed">[[historyDate]]</div> |
| + <template is="dom-if" if="{{!isNotSearchResult_(searchTerm)}}"> |
|
tsergeant
2016/02/02 02:57:36
Can you get rid of the dom-ifs by having a single
hsampson
2016/02/03 02:37:58
Done.
tsergeant
2016/02/04 02:28:49
Yay!
|
| + <div id="date-accessed"> |
| + {{searchResultTitle_(historyItems.length,searchTerm)}} |
|
tsergeant
2016/02/02 02:57:36
Nit: space after comma
hsampson
2016/02/03 02:37:58
Done.
|
| + </div> |
| + </template> |
| + <template is="dom-if" if="{{isNotSearchResult_(searchTerm)}}"> |
| + <div id="date-accessed">[[historyDate]]</div> |
| + </template> |
| <div id="history-item-list"> |
| <template is="dom-repeat" items="{{historyItems}}" |
| as="historyItem"> |
| @@ -55,7 +66,8 @@ |
| 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)}}"> |
| <div id="time-gap-separator"></div> |