Bug 1265181 - show used fonts like "Helvetica, Arial, Sans-serif" correctly in the composer font picker. r=jorgk a=rkent THUNDERBIRD451b1_2016042214_RELBRANCH
authoraceman <acelists@atlas.sk>
Sat, 16 Apr 2016 14:53:00 +0200
branchTHUNDERBIRD451b1_2016042214_RELBRANCH
changeset 26907 ff7bcfb79ba3ef10ede9e4d4cb8b80a376bf884b
parent 26906 8b709e2759eafc3791edb1e0cc5bbfd2eb94251c
child 26908 63a01b468dc5ff20ca12a3047b4f5c3cfe53f5fe
push id1850
push userclokep@gmail.com
push dateWed, 08 Mar 2017 19:29:12 +0000
treeherdercomm-esr52@028df196b2d9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjorgk, rkent
bugs1265181
Bug 1265181 - show used fonts like "Helvetica, Arial, Sans-serif" correctly in the composer font picker. r=jorgk a=rkent
editor/ui/composer/content/editorOverlay.xul
mail/components/compose/content/editorOverlay.xul
--- a/editor/ui/composer/content/editorOverlay.xul
+++ b/editor/ui/composer/content/editorOverlay.xul
@@ -414,28 +414,31 @@
                   value="tt"
                   type="radio"
                   observes="cmd_renderedHTMLEnabler"/>
         <menuseparator id="fontFaceMenuAfterGenericFontsSeparator"/>
         <menuitem id="menu_fontFaceHelvetica"
                   label="&fontHelvetica.label;"
                   accesskey="&fontHelvetica.accesskey;"
                   value="Helvetica, Arial, sans-serif"
+                  value_parsed="helvetica,arial,sans-serif"
                   type="radio"
                   observes="cmd_renderedHTMLEnabler"/>
         <menuitem id="menu_fontFaceTimes"
                   label="&fontTimes.label;"
                   accesskey="&fontTimes.accesskey;"
                   value="Times New Roman, Times, serif"
+                  value_parsed="times new roman,times,serif"
                   type="radio"
                   observes="cmd_renderedHTMLEnabler"/>
         <menuitem id="menu_fontFaceCourier"
                   label="&fontCourier.label;"
                   accesskey="&fontCourier.accesskey;"
                   value="Courier New, Courier, monospace"
+                  value_parsed="courier new,courier,monospace"
                   type="radio"
                   observes="cmd_renderedHTMLEnabler"/>
         <menuseparator id="fontFaceMenuAfterDefaultFontsSeparator"
                        class="fontFaceMenuAfterDefaultFonts"/>
         <menuseparator id="fontFaceMenuAfterUsedFontsSeparator"
                        class="fontFaceMenuAfterUsedFonts"
                        collapsed="true"/>
           <!-- Local font face items added here by initLocalFontFaceMenu() -->
@@ -1066,21 +1069,24 @@
             crop="right">
     <observes element="font-face-select-container" attribute="disabled"/>
     <observes element="cmd_fontFace" attribute="state" onbroadcast="onFontFaceChange(this.parentNode, 'cmd_fontFace')"/>
     <menupopup id="FontFacePopup" oncommand="doStatefulCommand('cmd_fontFace', event.target.value)">
       <menuitem id="toolbarmenu_fontVarWidth" label="&fontVarWidth.label;" value=""/>
       <menuitem id="toolbarmenu_fontFixedWidth" label="&fontFixedWidth.label;" value="tt"/>
       <menuseparator id="toolbarmenuAfterGenericFontsSeparator"/>
       <menuitem id="toolbarmenu_fontHelvetica" label="&fontHelvetica.label;"
-                value="Helvetica, Arial, sans-serif"/>
+                value="Helvetica, Arial, sans-serif"
+                value_parsed="helvetica,arial,sans-serif"/>
       <menuitem id="toolbarmenu_fontTimes" label="&fontTimes.label;"
-                value="Times New Roman, Times, serif"/>
+                value="Times New Roman, Times, serif"
+                value_parsed="times new roman,times,serif"/>
       <menuitem id="toolbarmenu_fontCourier" label="&fontCourier.label;"
-                value="Courier New, Courier, monospace"/>
+                value="Courier New, Courier, monospace"
+                value_parsed="courier new,courier,monospace"/>
       <menuseparator id="toolbarmenuAfterDefaultFontsSeparator"
                      class="fontFaceMenuAfterDefaultFonts"/>
       <menuseparator id="toolbarmenuAfterUsedFontsSeparator"
                      class="fontFaceMenuAfterUsedFonts"
                      hidden="true"/>
       <!-- Local font face items added here by initLocalFontFaceMenu() -->
     </menupopup>
   </menulist>
--- a/mail/components/compose/content/editorOverlay.xul
+++ b/mail/components/compose/content/editorOverlay.xul
@@ -282,28 +282,31 @@
                   value="tt"
                   type="radio"
                   observes="cmd_renderedHTMLEnabler"/>
         <menuseparator id="fontFaceMenuAfterGenericFontsSeparator"/>
         <menuitem id="menu_fontFaceHelvetica"
                   label="&fontHelvetica.label;"
                   accesskey="&fontHelvetica.accesskey;"
                   value="Helvetica, Arial, sans-serif"
+                  value_parsed="helvetica,arial,sans-serif"
                   type="radio"
                   observes="cmd_renderedHTMLEnabler"/>
         <menuitem id="menu_fontFaceTimes"
                   label="&fontTimes.label;"
                   accesskey="&fontTimes.accesskey;"
                   value="Times New Roman, Times, serif"
+                  value_parsed="times new roman,times,serif"
                   type="radio"
                   observes="cmd_renderedHTMLEnabler"/>
         <menuitem id="menu_fontFaceCourier"
                   label="&fontCourier.label;"
                   accesskey="&fontCourier.accesskey;"
                   value="Courier New, Courier, monospace"
+                  value_parsed="courier new,courier,monospace"
                   type="radio"
                   observes="cmd_renderedHTMLEnabler"/>
         <menuseparator id="fontFaceMenuAfterDefaultFontsSeparator"
                        class="fontFaceMenuAfterDefaultFonts"/>
         <menuseparator id="fontFaceMenuAfterUsedFontsSeparator"
                        class="fontFaceMenuAfterUsedFonts"
                        hidden="true"/>
           <!-- Local font face items added here by initLocalFontFaceMenu() -->
@@ -876,21 +879,24 @@
     <observes element="font-face-select-container" attribute="disabled"/>
     <observes element="cmd_fontFace" attribute="state" onbroadcast="onFontFaceChange(this.parentNode, 'cmd_fontFace')"/>
     <menupopup id="FontFacePopup"
                class="noIconPopup">
       <menuitem id="toolbarmenu_fontVarWidth" label="&fontVarWidth.label;" value=""/>
       <menuitem id="toolbarmenu_fontFixedWidth" label="&fontFixedWidth.label;" value="tt"/>
       <menuseparator id="toolbarmenuAfterGenericFontsSeparator"/>
       <menuitem id="toolbarmenu_fontHelvetica" label="&fontHelvetica.label;"
-                value="Helvetica, Arial, sans-serif"/>
+                value="Helvetica, Arial, sans-serif"
+                value_parsed="helvetica,arial,sans-serif"/>
       <menuitem id="toolbarmenu_fontTimes" label="&fontTimes.label;"
-                value="Times New Roman, Times, serif"/>
+                value="Times New Roman, Times, serif"
+                value_parsed="times new roman,times,serif"/>
       <menuitem id="toolbarmenu_fontCourier" label="&fontCourier.label;"
-                value="Courier New, Courier, monospace"/>
+                value="Courier New, Courier, monospace"
+                value_parsed="courier new,courier,monospace"/>
       <menuseparator id="toolbarmenuAfterDefaultFontsSeparator"
                      class="fontFaceMenuAfterDefaultFonts"/>
       <menuseparator id="toolbarmenuAfterUsedFontsSeparator"
                      class="fontFaceMenuAfterUsedFonts"
                      hidden="true"/>
       <!-- Local font face items added here by initLocalFontFaceMenu() -->
     </menupopup>
   </menulist>