Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(219)

Side by Side Diff: chrome/browser/resources/settings/settings_shared_css.html

Issue 2855273002: Make fixed size scrolling more like other lists. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « chrome/browser/resources/settings/search_engines_page/search_engines_list.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <link rel="import" href="chrome://resources/cr_elements/shared_style_css.html"> 1 <link rel="import" href="chrome://resources/cr_elements/shared_style_css.html">
2 <link rel="import" href="settings_vars_css.html"> 2 <link rel="import" href="settings_vars_css.html">
3 3
4 <!-- Common styles for Material Design settings. --> 4 <!-- Common styles for Material Design settings. -->
5 <dom-module id="settings-shared"> 5 <dom-module id="settings-shared">
6 <template> 6 <template>
7 <style include="cr-shared-style"> 7 <style include="cr-shared-style">
8 :host-context([dir=rtl]) button[is='paper-icon-button-light'] { 8 :host-context([dir=rtl]) button[is='paper-icon-button-light'] {
9 transform: scaleX(-1); /* Flip on the X axis (aka mirror). */ 9 transform: scaleX(-1); /* Flip on the X axis (aka mirror). */
10 } 10 }
(...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 221
222 /* This button has no ink ripple. */ 222 /* This button has no ink ripple. */
223 .list-button[is='action-link'] { 223 .list-button[is='action-link'] {
224 align-items: center; 224 align-items: center;
225 display: flex; 225 display: flex;
226 flex: 1; 226 flex: 1;
227 font-weight: 500; 227 font-weight: 500;
228 min-height: inherit; 228 min-height: inherit;
229 } 229 }
230 230
231 .fixed-height-list { 231 .fixed-height-list {
scottchen 2017/05/03 23:01:27 - Would fixed-height-container be the more suitabl
dschuyler 2017/05/08 20:35:29 I'm flexible on the class name, but I agree that i
hcarmona 2017/05/08 21:32:23 Sorry for forwarding before updating the CL. Moved
232 max-height: calc((var(--settings-row-min-height) + 232 max-height: calc((var(--settings-row-min-height) +
233 var(--settings-separator-height)) * 6); 233 var(--settings-separator-height)) * 6);
234 overflow-y: auto;
235 } 234 }
236 235
237 /* A row with two lines of text. Often the lower line will be .secondary. 236 /* A row with two lines of text. Often the lower line will be .secondary.
238 */ 237 */
239 .two-line { 238 .two-line {
240 min-height: var(--settings-row-two-line-min-height); 239 min-height: var(--settings-row-two-line-min-height);
241 } 240 }
242 241
243 /* A settings-box is a horizontal row of text or controls within a 242 /* A settings-box is a horizontal row of text or controls within a
244 * setting section (page or subpage). */ 243 * setting section (page or subpage). */
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
392 transform: rotate(-135deg); 391 transform: rotate(-135deg);
393 } 392 }
394 393
395 .column-header { 394 .column-header {
396 color: var(--paper-grey-600); 395 color: var(--paper-grey-600);
397 font-weight: 500; 396 font-weight: 500;
398 } 397 }
399 </style> 398 </style>
400 </template> 399 </template>
401 </dom-module> 400 </dom-module>
OLDNEW
« no previous file with comments | « chrome/browser/resources/settings/search_engines_page/search_engines_list.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698