Chromium Code Reviews Chromium Code Reviews
Issue 8079010:
Change history2 to allow multi-deletion without entering a separate mode. (Closed)
Base URL: svn://svn.chromium.org/chrome/trunk/src| Index: chrome/browser/resources/history2.html |
| diff --git a/chrome/browser/resources/history2.html b/chrome/browser/resources/history2.html |
| index 36bdfe63684d939f8199b7a3ac2a524e926ab30e..ade732ab6b9aa5f7abf641baa8d9f2270514447b 100644 |
| --- a/chrome/browser/resources/history2.html |
| +++ b/chrome/browser/resources/history2.html |
| @@ -42,20 +42,12 @@ |
| text-overflow: ellipsis; |
| width: 50%; |
| } |
| -#edit-button { |
| - text-align: right; |
| - overflow: hidden; |
| - white-space: nowrap; |
| - text-overflow: ellipsis; |
| - width: 50%; |
| -} |
| #editing-controls button { |
| - margin-top: 18px; |
| - margin-bottom: -8px; |
| + margin: 18px 0 -8px 0; |
| } |
| #results-display { |
| max-width: 740px; |
| - margin: 16px 4px 0 4px; |
| + margin: 16px 0 0 0; |
| } |
| .day { |
| color: #6a6a6a; |
| @@ -84,7 +76,7 @@ |
| list-style: none; |
| } |
| .gap { |
| - width: 15px; |
| + width: 35px; |
| -webkit-border-end: 1px solid #ddd; |
| height: 14px; |
| } |
| @@ -92,6 +84,7 @@ |
| overflow: auto; /* Make sure it's at least as large as its children. */ |
| } |
| .entry-box { |
| + cursor: default; |
| line-height: 1.6em; |
| display: -webkit-box; |
| @@ -128,6 +121,7 @@ html[dir=rtl] .entry-box { |
| width: 13px; |
| height: 13px; |
| -webkit-margin-start: 8px; |
| + -webkit-margin-end: 4px; |
| position: relative; |
| top: 2px; |
| } |
| @@ -147,6 +141,23 @@ html[dir='rtl'] .entry .domain { |
| color:#9a9a9a; |
| width: 90px; |
| } |
| +.entry input[type=checkbox] { |
| + position: relative; |
| + top: 1px; |
| +} |
| + |
| +/* Checkboxes are shown when checked, or when the entry is hovered over. */ |
| +.entry input[type=checkbox]:not(:checked) { |
| + visibility: hidden; |
|
arv (Not doing code reviews)
2011/10/04 23:26:30
also, :focused
Maybe use opacity instead and add
Patrick Dubroy
2011/10/05 12:47:45
Good point. Done.
|
| +} |
| +.entry-box:hover input[type=checkbox] { |
| + visibility: visible; |
| +} |
| + |
| +.entry-box:hover { |
| + background-color: #e4ecf7; |
| + border-radius: 2px; |
| +} |
| .entry .title { |
| -webkit-box-flex: 1; |
| overflow: hidden; |
| @@ -175,6 +186,10 @@ html[dir='rtl'] .entry .domain { |
| .entry .title > a:visted { |
| color: #11c; |
| } |
| +.fade-out-200 { |
|
arv (Not doing code reviews)
2011/10/04 23:26:30
200?
class names should not describe the renderin
Patrick Dubroy
2011/10/05 12:47:45
Done. I did that because the JS code explicitly re
|
| + opacity: 0; |
| + -webkit-transition: opacity 200ms; |
| +} |
| </style> |
| </head> |
| <body i18n-values=".style.fontFamily:fontfamily;.style.fontSize:fontsize"> |
| @@ -193,7 +208,12 @@ html[dir='rtl'] .entry .domain { |
| <tr><td id="results-summary"></td><td id="edit-button"><p></p></td></tr> |
| </table> |
| </div> |
| - <div id="editing-controls"></div> |
| + <div id="editing-controls"> |
| + <button id="clear-browsing-data" i18n-content="clearallhistory"></button> |
| + <button id="remove-selected" |
| + disabled="disabled" |
| + i18n-content="removeselected"></button> |
| + </div> |
| <div id="results-display"></div> |
| <div id="results-pagination"></div> |
| </div> |
