summaryrefslogtreecommitdiffstats
path: root/chrome/browser/resources/options
diff options
context:
space:
mode:
authorfinnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-12-14 10:08:38 +0000
committerfinnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-12-14 10:08:38 +0000
commit5cc1b9f1b79307c2420c63e9ba7aefaf8bf2980b (patch)
treeb4e72178306b80a7d54c022b8cc834c67f05dcb6 /chrome/browser/resources/options
parentbca22bb6d9b12c3370f05336a7cf896b7be11612 (diff)
downloadchromium_src-5cc1b9f1b79307c2420c63e9ba7aefaf8bf2980b.zip
chromium_src-5cc1b9f1b79307c2420c63e9ba7aefaf8bf2980b.tar.gz
chromium_src-5cc1b9f1b79307c2420c63e9ba7aefaf8bf2980b.tar.bz2
Reverting half the checkins that are suspected of breaking the world.
Revert 114382 - Don't display spelling options in "Language Settings" pref panel on OS X We're removing Hunspell integration in favor of only using the native OS spell checker on OS X, see crbug.com/69944 for more details. This CL modifieds the "Languages and Settings..." preference pane to remove options related to spelling on the Mac. Specific changes: * In .grd file: don't include spelling-related strings on Mac. * Fixed description for IDS_OPTIONS_SETTINGS_USE_THIS_FOR_SPELL_CHECKING in .grd file to not mention ChromeOS. * In "Under the Hood" changed button title that opens Languages pref panel to just "Language Settings..." on Mac rather than "Languages and spell-checker Settings..." * In the Language settings panel - remove the right pane and bottom checkbox on the Mac (these relate to dictionary download and spell checking). * Change the background color of the language list pane to white on Mac. You can see before/after screenshots of this change here: crbug.com/69944#c16 . BUG=69944 TEST="Preferences->Under the hood" should have a "Languages and Spell-checker Settings..." button on non-Mac platforms and the subsequent pref pane should not be modified by this CL. On Mac, the button should be titled "Language Settings..." and the panel that opens should lack the right pane and only contain options to pick and re-order languages - it should not contain options related to spell checking. Review URL: http://codereview.chromium.org/8917002 TBR=jeremy@chromium.org Review URL: http://codereview.chromium.org/8940001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114399 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/resources/options')
-rw-r--r--chrome/browser/resources/options/language_options.css10
-rw-r--r--chrome/browser/resources/options/language_options.html2
-rw-r--r--chrome/browser/resources/options/language_options.js10
3 files changed, 4 insertions, 18 deletions
diff --git a/chrome/browser/resources/options/language_options.css b/chrome/browser/resources/options/language_options.css
index 55b7aa8..c1dc7a4 100644
--- a/chrome/browser/resources/options/language_options.css
+++ b/chrome/browser/resources/options/language_options.css
@@ -39,16 +39,6 @@
width: 300px;
}
-/* On OS X we use the native OS spellchecker, so don't display the dictionary */
-/* pane. */
-html[os=mac] .language-options-left {
- background-color: white;
-}
-
-html[os=mac] .language-options-right {
- visibility: hidden;
-}
-
.language-options-lower-left {
-webkit-box-flex: 0;
-webkit-padding-start: 12px;
diff --git a/chrome/browser/resources/options/language_options.html b/chrome/browser/resources/options/language_options.html
index 46373c1..3375bd2 100644
--- a/chrome/browser/resources/options/language_options.html
+++ b/chrome/browser/resources/options/language_options.html
@@ -57,7 +57,7 @@
<div i18n-content="switch_input_methods_hint"></div>
<div i18n-content="select_previous_input_method_hint"></div>
</if>
-<if expr="not pp_ifdef('chromeos') and not is_macosx">
+<if expr="not pp_ifdef('chromeos')">
<div id="spell-check-option" class="checkbox">
<label>
<input id="enable-spell-check" pref="browser.enable_spellchecking"
diff --git a/chrome/browser/resources/options/language_options.js b/chrome/browser/resources/options/language_options.js
index 47674ed..452c61c 100644
--- a/chrome/browser/resources/options/language_options.js
+++ b/chrome/browser/resources/options/language_options.js
@@ -93,11 +93,8 @@ cr.define('options', function() {
$('auto-spell-correction-option').hidden = false;
// Handle spell check enable/disable.
- if (!cr.isMac) {
- Preferences.getInstance().addEventListener(
- this.enableSpellCheckPref,
- this.updateEnableSpellCheck_.bind(this));
- }
+ Preferences.getInstance().addEventListener(this.enableSpellCheckPref,
+ this.updateEnableSpellCheck_.bind(this));
}
// Listen to user clicks on the "Change touch keyboard settings..."
@@ -222,8 +219,7 @@ cr.define('options', function() {
this.updateSelectedLanguageName_(languageCode);
if (cr.isWindows || cr.isChromeOS)
this.updateUiLanguageButton_(languageCode);
- if (!cr.isMac)
- this.updateSpellCheckLanguageButton_(languageCode);
+ this.updateSpellCheckLanguageButton_(languageCode);
if (cr.isChromeOS)
this.updateInputMethodList_(languageCode);
this.updateLanguageListInAddLanguageOverlay_();