Commit 996bc57c authored by dubroy@chromium.org's avatar dubroy@chromium.org

History: Fix regression - unable to remove entries from search results.

In the switchover from history2, the ability to select multiple history search results
(using the checkboxes) was lost. This CL restores that ability, and fixes some style
issues with the search results page.

BUG=106394
TEST=Manual: open chrome://history, do a search, and ensure that you can select and
delete multiple items.

Review URL: http://codereview.chromium.org/8807001

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113044 0039d316-1c4b-4281-b951-d872f2087c98
parent eec0b33c
...@@ -92,11 +92,17 @@ html[dir=rtl] .entry-box { ...@@ -92,11 +92,17 @@ html[dir=rtl] .entry-box {
} }
.snippet { .snippet {
-webkit-margin-start: 90px; /* Align it with .domain. */ -webkit-box-orient: vertical;
-webkit-line-clamp: 2;
-webkit-margin-start: 110px;
clear: both; clear: both;
color: rgb(34, 34, 34);
display: -webkit-box;
font-size: 12px; font-size: 12px;
line-height: 1.6em; line-height: 1.6em;
margin-bottom: 12px; margin-bottom: 12px;
overflow: hidden;
text-overflow: ellipsis;
} }
.entry .domain { .entry .domain {
...@@ -134,9 +140,16 @@ html[dir='rtl'] .entry .title { ...@@ -134,9 +140,16 @@ html[dir='rtl'] .entry .title {
.entry .time { .entry .time {
color: #9a9a9a; color: #9a9a9a;
overflow: hidden;
text-overflow: ellipsis;
white-space: nowrap;
width: 90px; width: 90px;
} }
.search-results .time {
width: 110px;
}
.entry input[type=checkbox] { .entry input[type=checkbox] {
-webkit-margin-end: 6px; -webkit-margin-end: 6px;
-webkit-margin-start: 4px; -webkit-margin-start: 4px;
......
...@@ -163,7 +163,7 @@ Page.prototype.getResultDOM = function(searchResultFlag) { ...@@ -163,7 +163,7 @@ Page.prototype.getResultDOM = function(searchResultFlag) {
node.appendChild(entryBox); node.appendChild(entryBox);
if (searchResultFlag) { if (searchResultFlag) {
time.textContent = this.dateShort; time.appendChild(document.createTextNode(this.dateShort));
var snippet = createElementWithClassName('div', 'snippet'); var snippet = createElementWithClassName('div', 'snippet');
this.addHighlightedText_(snippet, this.addHighlightedText_(snippet,
this.snippet_, this.snippet_,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment