MD Settings: Polish for Languages main page

chrome/browser/resources/settings/languages_page/languages_page.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview 'settings-languages-page' is the settings page
  * for language and input method settings.
  */
 (function() {
 'use strict';
@@ skipping 90 matching lines @@
    * @return {boolean} True if |language| is first in the list of enabled
    *     languages. Used to hide the "Move up" option.
    * @private
    */
   isFirstLanguage_: function(language) {
     return language == this.languages.enabled[0];
   },
 
   /**
    * @param {!LanguageState} language
+   * @return {boolean} True if |language| is first or second in the list of
+   *     enabled languages. Used to hide the "Move to top" option.
+   * @private
+   */
+  isFirstOrSecondLanguage_: function(language) {
+    return this.languages.enabled.slice(0, 2).includes(language);
+  },
+
+  /**
+   * @param {!LanguageState} language
    * @return {boolean} True if |language| is last in the list of enabled
    *     languages. Used to hide the "Move down" option.
    * @private
    */
   isLastLanguage_: function(language) {
     return language == this.languages.enabled.slice(-1)[0];
   },
 
   /**
    * @param {!Object} change Polymer change object for languages.enabled.*.
@@ skipping 47 matching lines @@
    * @private
    */
   onUILanguageChange_: function(e) {
     if (e.target.checked) {
       this.languageHelper.setUILanguage(this.detailLanguage_.language.code);
     } else if (this.detailLanguage_.language.code ==
         this.languageHelper.getProspectiveUILanguage()) {
       // Reset the chosen UI language to the actual UI language.
       this.languageHelper.resetUILanguage();
     }
+    /** @type {!CrSharedMenuElement} */(this.$.menu.get()).closeMenu();
   },
 
    /**
    * @param {!chrome.languageSettingsPrivate.Language} language
    * @param {string} targetLanguageCode The default translate target language.
    * @return {boolean} True if the translate checkbox should be disabled.
    * @private
    */
   disableTranslateCheckbox_: function(language, targetLanguageCode) {
     if (!language.supportsTranslate)
       return true;
 
     return this.languageHelper.convertLanguageCodeForTranslate(language.code) ==
         targetLanguageCode;
   },
 
   /**
    * Handler for changes to the translate checkbox.
    * @param {!{target: !PaperCheckboxElement}} e
    * @private
    */
   onTranslateCheckboxChange_: function(e) {
     if (e.target.checked) {
       this.languageHelper.enableTranslateLanguage(
           this.detailLanguage_.language.code);
     } else {
       this.languageHelper.disableTranslateLanguage(
           this.detailLanguage_.language.code);
     }
+    /** @type {!CrSharedMenuElement} */(this.$.menu.get()).closeMenu();
   },
 
   /**
+   * Returns "complex" if the menu includes checkboxes, which should change the
+   * spacing of items and show a separator in the menu.
+   * @param {boolean} translateEnabled
+   * @return {string}
+   */
+  getMenuClass_: function(translateEnabled) {
+    if (translateEnabled || cr.isChromeOS || cr.isWindows)
+      return 'complex';
+    return '';
+  },
+
+  /**
+   * Moves the language to the top of the list.
+   * @private
+   */
+  onMoveToTopTap_: function() {
+    /** @type {!CrSharedMenuElement} */(this.$.menu.get()).closeMenu();
+    this.languageHelper.moveLanguageToFront(this.detailLanguage_.language.code);
+  },
+
+  /**
    * Moves the language up in the list.
    * @private
    */
   onMoveUpTap_: function() {
     /** @type {!CrSharedMenuElement} */(this.$.menu.get()).closeMenu();
     this.languageHelper.moveLanguage(this.detailLanguage_.language.code, -1);
   },
 
   /**
    * Moves the language down in the list.
@@ skipping 243 matching lines @@
   onRestartTap_: function() {
 <if expr="chromeos">
     settings.LifetimeBrowserProxyImpl.getInstance().signOutAndRestart();
 </if>
 <if expr="not chromeos">
     settings.LifetimeBrowserProxyImpl.getInstance().restart();
 </if>
   },
 });
 })();