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

Unified Diff: chrome/browser/resources/settings/appearance_page/appearance_fonts_page.html

Issue 2738333004: [MD settings] set pref to valid slider value (Closed)
Patch Set: min font size Created 3 years, 9 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/appearance_page/appearance_fonts_page.html
diff --git a/chrome/browser/resources/settings/appearance_page/appearance_fonts_page.html b/chrome/browser/resources/settings/appearance_page/appearance_fonts_page.html
index 53654b05069e6e44bf1a3c9ee8ebdead5643cc57..a5bcb3077942667842b9efeca5d22ac1f4c1335b 100644
--- a/chrome/browser/resources/settings/appearance_page/appearance_fonts_page.html
+++ b/chrome/browser/resources/settings/appearance_page/appearance_fonts_page.html
@@ -22,10 +22,12 @@
<div class="settings-box">
<div class="start">$i18n{minimumFont}</div>
<div class="list-item"
- style$="[[computeStyle_(
+ style$="[[computeMinimumStyle_(
Dan Beam 2017/03/14 03:27:19 btw, you can now do style$="font-family: [[comput
dschuyler 2017/03/14 18:55:31 Done.
prefs.webkit.webprefs.minimum_font_size.value,
prefs.webkit.webprefs.fonts.standard.Zyyy.value)]]">
- <span>[[prefs.webkit.webprefs.minimum_font_size.value]]</span>
+ <span>[[
+ computeMinimumFontSize_(
+ prefs.webkit.webprefs.minimum_font_size.value)]]</span>
: $i18n{quickBrownFox}
</div>
<cr-slider id="minimumSizeSlider"

Powered by Google App Engine
This is Rietveld 408576698