Bug 1430551 - Make editor use <div> as defaultParagraphSeparator even in release channel r?m_kato draft
authorMasayuki Nakano <masayuki@d-toybox.com>
Mon, 12 Feb 2018 00:31:44 +0900
changeset 754771 04ee9af5e06b2000d15dc84c8b539f43d9ce816a
parent 754770 389c0609547bc2e8e7679cf870cebb6c526a3c80
child 754772 cb56fa69288846ee2a4b57e3b97bddd49364a3ca
child 754774 fe9521f4fef57a7a5f345a3a5a09907a8f4a769a
push id98989
push usermasayuki@d-toybox.com
push dateWed, 14 Feb 2018 07:43:07 +0000
reviewersm_kato
bugs1430551
milestone60.0a1
Bug 1430551 - Make editor use <div> as defaultParagraphSeparator even in release channel r?m_kato No Nightly testers don't report new compatibility issue. Additionally, if we make Firefox use <div> as defaultParagraphSeparator in release build, web services may stop supporting our current behavior quickly because they can get rid of hack for us. Therefore, we should do this in the cycle of Gecko 60 which is next ESR. If we did this later, ESR users may have become not to be able to use existing web services suddenly immediately after we did this in 61 or 62. We should avoid this bad scenario. MozReview-Commit-ID: 7Um79Ky7n8i
modules/libpref/init/all.js
--- a/modules/libpref/init/all.js
+++ b/modules/libpref/init/all.js
@@ -1258,21 +1258,17 @@ pref("extensions.spellcheck.inline.max-m
 // belong in comm-central/editor/ui/composer.js
 
 pref("editor.use_custom_colors", false);
 pref("editor.singleLine.pasteNewlines",      2);
 pref("editor.use_css",                       false);
 pref("editor.css.default_length_unit",       "px");
 pref("editor.resizing.preserve_ratio",       true);
 pref("editor.positioning.offset",            0);
-#ifdef EARLY_BETA_OR_EARLIER
 pref("editor.use_div_for_default_newlines",  true);
-#else
-pref("editor.use_div_for_default_newlines",  false);
-#endif
 
 // Scripts & Windows prefs
 pref("dom.disable_beforeunload",            false);
 pref("dom.disable_window_flip",             false);
 pref("dom.disable_window_move_resize",      false);
 
 pref("dom.disable_window_open_feature.titlebar",    false);
 pref("dom.disable_window_open_feature.close",       false);