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

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

Issue 2817243003: [MD settings] observe changes to default font size in Appearance (Closed)
Patch Set: Created 3 years, 8 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 7b2fc2ff8fc434e0e15e579b5b3ae40727ef52d2..72fac987255be102e0434689abb2f4497b8f0075 100644
--- a/chrome/browser/resources/settings/appearance_page/appearance_fonts_page.html
+++ b/chrome/browser/resources/settings/appearance_page/appearance_fonts_page.html
@@ -111,13 +111,11 @@
</div>
<div class="list-item"
style="
- font-size:[[prefs.webkit.webprefs.default_font_size.value]]px;
+ font-size:
+ [[prefs.webkit.webprefs.default_fixed_font_size.value]]px;
dschuyler 2017/04/15 01:27:57 This is part of the 'also fixed' an not the primar
font-family:
'[[prefs.webkit.webprefs.fonts.fixed.Zyyy.value]]';">
- <span>
- [[prefs.webkit.webprefs.default_font_size.value]]:
- $i18n{quickBrownFox}
- </span>
+ $i18n{quickBrownFox}
dschuyler 2017/04/15 01:27:57 So it looks strange to show that this is 3px small
michaelpg 2017/04/17 20:06:28 SGTM. It would be more consistent to remove the pr
dschuyler 2017/04/17 21:26:11 Acknowledged.
</div>
</div>
<template is="dom-if" if="[[!isGuest_]]">

Powered by Google App Engine
This is Rietveld 408576698