Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 <link rel="import" href="chrome://resources/html/polymer.html"> | 1 <link rel="import" href="chrome://resources/html/polymer.html"> |
| 2 <link rel="import" href="chrome://resources/html/i18n_behavior.html"> | 2 <link rel="import" href="chrome://resources/html/i18n_behavior.html"> |
| 3 <link rel="import" href="chrome://resources/html/web_ui_listener_behavior.html"> | 3 <link rel="import" href="chrome://resources/html/web_ui_listener_behavior.html"> |
| 4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper -icon-button-light.html"> | 4 <link rel="import" href="chrome://resources/polymer/v1_0/paper-icon-button/paper -icon-button-light.html"> |
| 5 <link rel="import" href="fonts_browser_proxy.html"> | 5 <link rel="import" href="fonts_browser_proxy.html"> |
| 6 <link rel="import" href="../controls/settings_slider.html"> | 6 <link rel="import" href="../controls/settings_slider.html"> |
| 7 <link rel="import" href="../controls/settings_dropdown_menu.html"> | 7 <link rel="import" href="../controls/settings_dropdown_menu.html"> |
| 8 <link rel="import" href="../i18n_setup.html"> | 8 <link rel="import" href="../i18n_setup.html"> |
| 9 <link rel="import" href="../settings_shared_css.html"> | 9 <link rel="import" href="../settings_shared_css.html"> |
| 10 | 10 |
| (...skipping 93 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 104 </div> | 104 </div> |
| 105 <div class="list-frame"> | 105 <div class="list-frame"> |
| 106 <div class="list-item"> | 106 <div class="list-item"> |
| 107 <settings-dropdown-menu class="start" label="$i18n{fixedWidthFont}" | 107 <settings-dropdown-menu class="start" label="$i18n{fixedWidthFont}" |
| 108 pref="{{prefs.webkit.webprefs.fonts.fixed.Zyyy}}" | 108 pref="{{prefs.webkit.webprefs.fonts.fixed.Zyyy}}" |
| 109 menu-options="[[fontOptions_]]"> | 109 menu-options="[[fontOptions_]]"> |
| 110 </settings-dropdown-menu> | 110 </settings-dropdown-menu> |
| 111 </div> | 111 </div> |
| 112 <div class="list-item" | 112 <div class="list-item" |
| 113 style=" | 113 style=" |
| 114 font-size:[[prefs.webkit.webprefs.default_font_size.value]]px; | 114 font-size: |
| 115 [[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
| |
| 115 font-family: | 116 font-family: |
| 116 '[[prefs.webkit.webprefs.fonts.fixed.Zyyy.value]]';"> | 117 '[[prefs.webkit.webprefs.fonts.fixed.Zyyy.value]]';"> |
| 117 <span> | 118 $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.
| |
| 118 [[prefs.webkit.webprefs.default_font_size.value]]: | |
| 119 $i18n{quickBrownFox} | |
| 120 </span> | |
| 121 </div> | 119 </div> |
| 122 </div> | 120 </div> |
| 123 <template is="dom-if" if="[[!isGuest_]]"> | 121 <template is="dom-if" if="[[!isGuest_]]"> |
| 124 <div class="settings-box two-line" id="advancedButton" | 122 <div class="settings-box two-line" id="advancedButton" |
| 125 on-tap="openAdvancedExtension_" actionable> | 123 on-tap="openAdvancedExtension_" actionable> |
| 126 <div class="start"> | 124 <div class="start"> |
| 127 $i18n{advancedFontSettings} | 125 $i18n{advancedFontSettings} |
| 128 <div class="secondary" id="advancedButtonSublabel"> | 126 <div class="secondary" id="advancedButtonSublabel"> |
| 129 [[advancedExtensionSublabel_]] | 127 [[advancedExtensionSublabel_]] |
| 130 </div> | 128 </div> |
| 131 </div> | 129 </div> |
| 132 <button class="icon-external" is="paper-icon-button-light" | 130 <button class="icon-external" is="paper-icon-button-light" |
| 133 aria-label="$i18n{advancedFontSettings}" | 131 aria-label="$i18n{advancedFontSettings}" |
| 134 aria-describedby="advancedButtonSublabel"></button> | 132 aria-describedby="advancedButtonSublabel"></button> |
| 135 </div> | 133 </div> |
| 136 </template> | 134 </template> |
| 137 </template> | 135 </template> |
| 138 <script src="appearance_fonts_page.js"></script> | 136 <script src="appearance_fonts_page.js"></script> |
| 139 </dom-module> | 137 </dom-module> |
| OLD | NEW |