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| OLD | NEW |
|---|---|
| 1 <!DOCTYPE HTML> | 1 <!DOCTYPE HTML> |
| 2 <html i18n-values="dir:textdirection;"> | 2 <html i18n-values="dir:textdirection;"> |
| 3 <head> | 3 <head> |
| 4 <meta charset="utf-8"> | 4 <meta charset="utf-8"> |
| 5 <include src="content_security_policy.html"/> | 5 <include src="content_security_policy.html"/> |
| 6 <title i18n-content="title"></title> | 6 <title i18n-content="title"></title> |
| 7 <link rel="icon" href="../../app/theme/history_favicon.png"> | 7 <link rel="icon" href="../../app/theme/history_favicon.png"> |
| 8 | 8 |
| 9 <link rel="stylesheet" href="chrome://resources/css/menu.css"> | 9 <link rel="stylesheet" href="chrome://resources/css/menu.css"> |
| 10 | 10 |
| (...skipping 24 matching lines...) Expand all Loading... | |
| 35 } | 35 } |
| 36 #results-separator table { | 36 #results-separator table { |
| 37 width: 100%; | 37 width: 100%; |
| 38 } | 38 } |
| 39 #results-summary { | 39 #results-summary { |
| 40 overflow: hidden; | 40 overflow: hidden; |
| 41 white-space: nowrap; | 41 white-space: nowrap; |
| 42 text-overflow: ellipsis; | 42 text-overflow: ellipsis; |
| 43 width: 50%; | 43 width: 50%; |
| 44 } | 44 } |
| 45 #edit-button { | |
| 46 text-align: right; | |
| 47 overflow: hidden; | |
| 48 white-space: nowrap; | |
| 49 text-overflow: ellipsis; | |
| 50 width: 50%; | |
| 51 } | |
| 52 #editing-controls button { | 45 #editing-controls button { |
| 53 margin-top: 18px; | 46 margin: 18px 0 -8px 0; |
| 54 margin-bottom: -8px; | |
| 55 } | 47 } |
| 56 #results-display { | 48 #results-display { |
| 57 max-width: 740px; | 49 max-width: 740px; |
| 58 margin: 16px 4px 0 4px; | 50 margin: 16px 0 0 0; |
| 59 } | 51 } |
| 60 .day { | 52 .day { |
| 61 color: #6a6a6a; | 53 color: #6a6a6a; |
| 62 font-weight: bold; | 54 font-weight: bold; |
| 63 margin: 0 0 4px 0; | 55 margin: 0 0 4px 0; |
| 64 text-transform: uppercase; | 56 text-transform: uppercase; |
| 65 font-size: 13px; | 57 font-size: 13px; |
| 66 } | 58 } |
| 67 .edit-button { | 59 .edit-button { |
| 68 display: inline; | 60 display: inline; |
| 69 -webkit-appearance: none; | 61 -webkit-appearance: none; |
| 70 background: none; | 62 background: none; |
| 71 border: 0; | 63 border: 0; |
| 72 color: blue; /* -webkit-link makes it purple :'( */ | 64 color: blue; /* -webkit-link makes it purple :'( */ |
| 73 cursor: pointer; | 65 cursor: pointer; |
| 74 text-decoration: underline; | 66 text-decoration: underline; |
| 75 padding:0px 9px; | 67 padding:0px 9px; |
| 76 display:inline-block; | 68 display:inline-block; |
| 77 font:inherit; | 69 font:inherit; |
| 78 } | 70 } |
| 79 .gap, | 71 .gap, |
| 80 .entry, | 72 .entry, |
| 81 .no-entries { | 73 .no-entries { |
| 82 margin: 0; | 74 margin: 0; |
| 83 padding: 0; | 75 padding: 0; |
| 84 list-style: none; | 76 list-style: none; |
| 85 } | 77 } |
| 86 .gap { | 78 .gap { |
| 87 width: 15px; | 79 width: 35px; |
| 88 -webkit-border-end: 1px solid #ddd; | 80 -webkit-border-end: 1px solid #ddd; |
| 89 height: 14px; | 81 height: 14px; |
| 90 } | 82 } |
| 91 .entry { | 83 .entry { |
| 92 overflow: auto; /* Make sure it's at least as large as its children. */ | 84 overflow: auto; /* Make sure it's at least as large as its children. */ |
| 93 } | 85 } |
| 94 .entry-box { | 86 .entry-box { |
| 87 cursor: default; | |
| 95 line-height: 1.6em; | 88 line-height: 1.6em; |
| 96 | 89 |
| 97 display: -webkit-box; | 90 display: -webkit-box; |
| 98 -webkit-box-orient: horizontal; | 91 -webkit-box-orient: horizontal; |
| 99 max-width: 100%; /* Don't allow it to be bigger than its parent... */ | 92 max-width: 100%; /* Don't allow it to be bigger than its parent... */ |
| 100 float: left; /* ..but make the box shrink to fit its content. */ | 93 float: left; /* ..but make the box shrink to fit its content. */ |
| 101 overflow: hidden; | 94 overflow: hidden; |
| 102 } | 95 } |
| 103 html[dir=rtl] .entry-box { | 96 html[dir=rtl] .entry-box { |
| 104 float: right; /* To make the box shrink to fit its content. */ | 97 float: right; /* To make the box shrink to fit its content. */ |
| (...skipping 16 matching lines...) Expand all Loading... | |
| 121 background-repeat: no-repeat; | 114 background-repeat: no-repeat; |
| 122 background-position-y: center; | 115 background-position-y: center; |
| 123 } | 116 } |
| 124 .drop-down { | 117 .drop-down { |
| 125 background: #fff -webkit-canvas(drop-down-arrow) no-repeat center center; | 118 background: #fff -webkit-canvas(drop-down-arrow) no-repeat center center; |
| 126 border: 1px solid hsl(214, 91%, 85%); | 119 border: 1px solid hsl(214, 91%, 85%); |
| 127 border-radius: 2px; | 120 border-radius: 2px; |
| 128 width: 13px; | 121 width: 13px; |
| 129 height: 13px; | 122 height: 13px; |
| 130 -webkit-margin-start: 8px; | 123 -webkit-margin-start: 8px; |
| 124 -webkit-margin-end: 4px; | |
| 131 position: relative; | 125 position: relative; |
| 132 top: 2px; | 126 top: 2px; |
| 133 } | 127 } |
| 134 .drop-down:hover { | 128 .drop-down:hover { |
| 135 border-color: #6A86DE; | 129 border-color: #6A86DE; |
| 136 background-image: -webkit-canvas(drop-down-arrow-hover); | 130 background-image: -webkit-canvas(drop-down-arrow-hover); |
| 137 } | 131 } |
| 138 .drop-down[menu-shown], .drop-down:focus { | 132 .drop-down[menu-shown], .drop-down:focus { |
| 139 border-color: #6A86DE; | 133 border-color: #6A86DE; |
| 140 background-color: #6A86DE; | 134 background-color: #6A86DE; |
| 141 background-image: -webkit-canvas(drop-down-arrow-active); | 135 background-image: -webkit-canvas(drop-down-arrow-active); |
| 142 } | 136 } |
| 143 html[dir='rtl'] .entry .domain { | 137 html[dir='rtl'] .entry .domain { |
| 144 background-position-x: right; | 138 background-position-x: right; |
| 145 } | 139 } |
| 146 .entry .time { | 140 .entry .time { |
| 147 color:#9a9a9a; | 141 color:#9a9a9a; |
| 148 width: 90px; | 142 width: 90px; |
| 149 } | 143 } |
| 144 .entry input[type=checkbox] { | |
| 145 position: relative; | |
| 146 top: 1px; | |
| 147 } | |
| 148 | |
| 149 /* Checkboxes are shown when checked, or when the entry is hovered over. */ | |
| 150 .entry input[type=checkbox]:not(:checked) { | |
| 151 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.
| |
| 152 } | |
| 153 .entry-box:hover input[type=checkbox] { | |
| 154 visibility: visible; | |
| 155 } | |
| 156 | |
| 157 .entry-box:hover { | |
| 158 background-color: #e4ecf7; | |
| 159 border-radius: 2px; | |
| 160 } | |
| 150 .entry .title { | 161 .entry .title { |
| 151 -webkit-box-flex: 1; | 162 -webkit-box-flex: 1; |
| 152 overflow: hidden; | 163 overflow: hidden; |
| 153 white-space: nowrap; | 164 white-space: nowrap; |
| 154 text-overflow: ellipsis; | 165 text-overflow: ellipsis; |
| 155 } | 166 } |
| 156 .entry .title > .starred { | 167 .entry .title > .starred { |
| 157 background:url('shared/images/star_small.png'); | 168 background:url('shared/images/star_small.png'); |
| 158 background-repeat:no-repeat; | 169 background-repeat:no-repeat; |
| 159 display:inline-block; | 170 display:inline-block; |
| 160 -webkit-margin-start: 4px; | 171 -webkit-margin-start: 4px; |
| 161 width:11px; | 172 width:11px; |
| 162 height:11px; | 173 height:11px; |
| 163 } | 174 } |
| 164 .entry .title > a { | 175 .entry .title > a { |
| 165 color: #11c; | 176 color: #11c; |
| 166 text-decoration: none; | 177 text-decoration: none; |
| 167 } | 178 } |
| 168 .entry .title > a.to-be-removed { | 179 .entry .title > a.to-be-removed { |
| 169 text-decoration: line-through; | 180 text-decoration: line-through; |
| 170 } | 181 } |
| 171 .entry .title > a:hover { | 182 .entry .title > a:hover { |
| 172 text-decoration: underline; | 183 text-decoration: underline; |
| 173 } | 184 } |
| 174 /* Since all history links are visited, we can make them blue. */ | 185 /* Since all history links are visited, we can make them blue. */ |
| 175 .entry .title > a:visted { | 186 .entry .title > a:visted { |
| 176 color: #11c; | 187 color: #11c; |
| 177 } | 188 } |
| 189 .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
| |
| 190 opacity: 0; | |
| 191 -webkit-transition: opacity 200ms; | |
| 192 } | |
| 178 </style> | 193 </style> |
| 179 </head> | 194 </head> |
| 180 <body i18n-values=".style.fontFamily:fontfamily;.style.fontSize:fontsize"> | 195 <body i18n-values=".style.fontFamily:fontfamily;.style.fontSize:fontsize"> |
| 181 <div class="header"> | 196 <div class="header"> |
| 182 <a id="history-section" href=""> | 197 <a id="history-section" href=""> |
| 183 <img src="shared/images/history_section.png" | 198 <img src="shared/images/history_section.png" |
| 184 width="67" height="67" class="logo" border="0"></a> | 199 width="67" height="67" class="logo" border="0"></a> |
| 185 <form id="search-form" method="post" action="" class="form"> | 200 <form id="search-form" method="post" action="" class="form"> |
| 186 <input type="text" name="term" id="term"> | 201 <input type="text" name="term" id="term"> |
| 187 <input type="submit" name="submit" i18n-values="value:searchbutton"> | 202 <input type="submit" name="submit" i18n-values="value:searchbutton"> |
| 188 </form> | 203 </form> |
| 189 </div> | 204 </div> |
| 190 <div class="main"> | 205 <div class="main"> |
| 191 <div id="results-separator"> | 206 <div id="results-separator"> |
| 192 <table border="0" cellPadding="0" cellSpacing="0"> | 207 <table border="0" cellPadding="0" cellSpacing="0"> |
| 193 <tr><td id="results-summary"></td><td id="edit-button"><p></p></td></tr> | 208 <tr><td id="results-summary"></td><td id="edit-button"><p></p></td></tr> |
| 194 </table> | 209 </table> |
| 195 </div> | 210 </div> |
| 196 <div id="editing-controls"></div> | 211 <div id="editing-controls"> |
| 212 <button id="clear-browsing-data" i18n-content="clearallhistory"></button> | |
| 213 <button id="remove-selected" | |
| 214 disabled="disabled" | |
| 215 i18n-content="removeselected"></button> | |
| 216 </div> | |
| 197 <div id="results-display"></div> | 217 <div id="results-display"></div> |
| 198 <div id="results-pagination"></div> | 218 <div id="results-pagination"></div> |
| 199 </div> | 219 </div> |
| 200 <div class="footer"> | 220 <div class="footer"> |
| 201 </div> | 221 </div> |
| 202 | 222 |
| 203 <menu id="action-menu"> | 223 <menu id="action-menu"> |
| 204 <button id="more-from-site" i18n-content="moreFromSite"></button> | 224 <button id="more-from-site" i18n-content="moreFromSite"></button> |
| 205 <button id="remove-page" i18n-content="removeFromHistory"></button> | 225 <button id="remove-page" i18n-content="removeFromHistory"></button> |
| 206 </menu> | 226 </menu> |
| 207 | 227 |
| 208 <script src="chrome://resources/js/i18n_template.js"></script> | 228 <script src="chrome://resources/js/i18n_template.js"></script> |
| 209 <script src="chrome://resources/js/i18n_process.js"></script> | 229 <script src="chrome://resources/js/i18n_process.js"></script> |
| 210 </body> | 230 </body> |
| 211 </html> | 231 </html> |
| OLD | NEW |
