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

Unified Diff: chrome/browser/resources/settings/controls/settings_toggle_button.html

Issue 2661533002: MD Settings: fix settings-toggle-button label text overflow. (Closed)
Patch Set: do text-elide and no-min-width in a cleaner way Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/settings/controls/settings_toggle_button.html
diff --git a/chrome/browser/resources/settings/controls/settings_toggle_button.html b/chrome/browser/resources/settings/controls/settings_toggle_button.html
index fc96b83bbdfbedd3ca22e84ca593f96f75f17acf..add7e52112f991a3e92d1f5697c05dd498a709df 100644
--- a/chrome/browser/resources/settings/controls/settings_toggle_button.html
+++ b/chrome/browser/resources/settings/controls/settings_toggle_button.html
@@ -6,6 +6,16 @@
<dom-module id="settings-toggle-button">
<template>
<style include="settings-shared">
+ :host([elide-label]),
+ :host([elide-label]) #outerRow,
+ :host([elide-label]) #outerRow > div.flex {
+ min-width: 0;
+ }
+
+ :host([elide-label]) .label {
+ @apply(--settings-text-elide);
+ }
+
#outerRow {
align-items: center;
display: flex;
@@ -29,8 +39,8 @@
<div id="outerRow" noSubLabel$="[[!subLabel]]">
<div class="flex" on-tap="onLabelWrapperTap_"
actionable$="[[!controlDisabled_(disabled, pref)]]">
- <div>[[label]]</div>
- <div class="secondary">[[subLabel]]</div>
+ <div class="label">[[label]]</div>
+ <div class="secondary label">[[subLabel]]</div>
</div>
<content selector=".more-actions"></content>
<template is="dom-if" if="[[pref.controlledBy]]">

Powered by Google App Engine
This is Rietveld 408576698