Chromium Code Reviews Chromium Code Reviews
Issue 15969014:
History: Update managed user history page (Closed)
Base URL: svn://svn.chromium.org/chrome/trunk/src| Index: chrome/browser/resources/history/history.html |
| diff --git a/chrome/browser/resources/history/history.html b/chrome/browser/resources/history/history.html |
| index 14e35f75fb7dcc3745d555c1f952440a865e2eba..5608015ee1bebe45967d406b4c61b9f66e53c068 100644 |
| --- a/chrome/browser/resources/history/history.html |
| +++ b/chrome/browser/resources/history/history.html |
| @@ -61,40 +61,48 @@ |
| <body i18n-values=".style.fontFamily:fontfamily"> |
| </if> |
| -<div class="page"> |
| +<div id="page" class="page"> |
|
Bernhard Bauer
2013/06/10 12:33:27
Is there ever only going to be one page?
Sergiu
2013/06/10 13:18:41
Renamed to "history-page". I need to get that elem
|
| <header> |
| <h1 i18n-content="history"></h1> |
| <div id="search-form" class="search-field-container"> |
| <input type="search" id="search-field" required> |
| <input type="submit" id="search-button" i18n-values="value:searchButton"> |
| </div> |
| - </header> |
| - |
| - <div id="filter-controls" hidden> |
| - <button id="range-today" i18n-content="rangeToday" disabled> |
| - </button> |
| - <button id="range-previous" disabled> |
| - <img src="../disclosure_triangle_small.png" |
| - i18n-values="alt:rangePrevious"> |
| - </button> |
| - <button id="range-next" disabled> |
| - <img src="../disclosure_triangle_small.png" i18n-values="alt:rangeNext"> |
| - </button> |
| - <div id="display-filter-controls"> |
| - <label> |
| - <input type="checkbox" id="group-by-domain" name="group-by-domain"> |
| - <span i18n-content="groupByDomainLabel"></span> |
| - </label> |
| - <label> |
| - <span i18n-content="rangeLabel"></span> |
| - <select id="timeframe-filter"> |
| - <option i18n-content="rangeAllTime" value="0"></option> |
| - <option i18n-content="rangeWeek" value="1"></option> |
| - <option i18n-content="rangeMonth" value="2"></option> |
| - </select> |
| - </label> |
| + <div id="filter-controls" hidden> |
| + <button id="range-today" i18n-content="rangeToday" disabled></button> |
| + <button id="range-previous" disabled> |
| + <img src="../disclosure_triangle_small.png" |
|
Bernhard Bauer
2013/06/10 12:33:27
I thought we usually put images in CSS?
Sergiu
2013/06/10 13:18:41
I remember discussing with Patrick (related CL whe
Bernhard Bauer
2013/06/10 13:30:28
Not sure what you mean; Pat said you should use a
Sergiu
2013/06/10 13:40:45
Yes, at that time he was referring to using "backg
Bernhard Bauer
2013/06/10 14:18:27
Hm. The CSS property has the advantage that you ca
Sergiu
2013/06/10 16:11:27
Done, see the new patch.
|
| + i18n-values="alt:rangePrevious"> |
| + </button><button id="range-next" disabled> |
| + <img src="../disclosure_triangle_small.png" i18n-values="alt:rangeNext"> |
| + </button> |
| + <div id="display-filter-controls"> |
| + <div class="display-filter-button"> |
| + <label for="timeframe-filter-all"> |
| + <input id="timeframe-filter-all" type="radio" |
| + name="timeframe-filter" value="0" checked> |
| + <span i18n-content="rangeAllTime" class="first-button-component"> |
| + </span> |
| + </label> |
| + </div> |
| + <div class="display-filter-button"> |
| + <label for="timeframe-filter-week"> |
| + <input id="timeframe-filter-week" type="radio" |
| + name="timeframe-filter" value="1"> |
| + <span i18n-content="rangeWeek"></span> |
| + </label> |
| + </div> |
| + <div class="display-filter-button"> |
| + <label for="timeframe-filter-month"> |
| + <input id="timeframe-filter-month" type="radio" |
| + name="timeframe-filter" value="2"> |
| + <span i18n-content="rangeMonth" class="last-button-component"> |
| + </span> |
| + </label> |
| + </div> |
| + </div> |
| </div> |
| - </div> |
| + </header> |
| <if expr="not is_android"> |
| <div id="other-devices" class="other-devices"></div> |
| </if> |
