User Manager MD User Pods

ui/login/account_picker/user_pod_row.js

 // Copyright 2014 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 User pod row implementation.
  */
 
 cr.define('login', function() {
   /**
@@ skipping 10 matching lines @@
    * @const
    */
   var MARGIN_BY_COLUMNS = [undefined, 40, 40, 40, 40, 40, 12];
 
   /**
    * Mapping between number of columns in the desktop pod-row and margin
    * between user pods for such layout.
    * @type {Array<number>}
    * @const
    */
-  var DESKTOP_MARGIN_BY_COLUMNS = [undefined, 15, 15, 15, 15, 15, 15];
+  var DESKTOP_MARGIN_BY_COLUMNS = [undefined, 32, 32, 32, 32, 32, 32];
 
   /**
    * Maximal number of columns currently supported by pod-row.
    * @type {number}
    * @const
    */
   var MAX_NUMBER_OF_COLUMNS = 6;
 
   /**
    * Maximal number of rows if sign-in banner is displayed alonside.
    * @type {number}
    * @const
    */
   var MAX_NUMBER_OF_ROWS_UNDER_SIGNIN_BANNER = 2;
 
   /**
    * Variables used for pod placement processing. Width and height should be
    * synced with computed CSS sizes of pods.
    */
-  var POD_WIDTH = 180;
+  var CROS_POD_WIDTH = 180;
+  var DESKTOP_POD_WIDTH = 160;
   var PUBLIC_EXPANDED_BASIC_WIDTH = 500;
   var PUBLIC_EXPANDED_ADVANCED_WIDTH = 610;
   var CROS_POD_HEIGHT = 213;
-  var DESKTOP_POD_HEIGHT = 226;
+  var DESKTOP_POD_HEIGHT = 200;
   var POD_ROW_PADDING = 10;
-  var DESKTOP_ROW_PADDING = 15;
+  var DESKTOP_ROW_PADDING = 32;
   var CUSTOM_ICON_CONTAINER_SIZE = 40;
 
   /**
    * Minimal padding between user pod and virtual keyboard.
    * @type {number}
    * @const
    */
   var USER_POD_KEYBOARD_MIN_PADDING = 20;
 
   /**
@@ skipping 1431 matching lines @@
 
       var is_synced_user = this.user.emailAddress !== "";
       // Write total number if all statistics are loaded.
       if (num_stats_loaded === Object.keys(stats_elements).length) {
         if (!total_count) {
           this.classList.add('has-no-stats');
           var message = loadTimeData.getString(
               is_synced_user ? 'removeUserWarningTextSyncNoStats' :
                                'removeUserWarningTextNonSyncNoStats');
           this.updateRemoveWarningDialogSetMessage_(this.user.profilePath,
-                                                    message);
+                                                    message, is_synced_user);
         } else {
           window.updateRemoveWarningDialogSetMessage =
               this.updateRemoveWarningDialogSetMessage_.bind(this);
           chrome.send('getRemoveWarningDialogMessage',[{
               profilePath: this.user.profilePath,
               isSyncedUser: is_synced_user,
               hasErrors: hasErrors,
               totalCount: total_count
           }]);
         }
       } else if (isInitial) {
         if (!this.user.isProfileLoaded) {
           message = loadTimeData.getString(
               is_synced_user ? 'removeUserWarningTextSyncNoStats' :
                                'removeUserWarningTextNonSyncNoStats');
           this.updateRemoveWarningDialogSetMessage_(this.user.profilePath,
-                                                    message);
+                                                    message, is_synced_user);
         } else {
           message = loadTimeData.getString(
               is_synced_user ? 'removeUserWarningTextSyncCalculating' :
                                'removeUserWarningTextNonSyncCalculating');
           substitute = loadTimeData.getString(
               'removeUserWarningTextCalculating');
           this.updateRemoveWarningDialogSetMessage_(this.user.profilePath,
-                                                    message, substitute);
+                                                    message, is_synced_user,
+                                                    substitute);
         }
       }
     },
 
     /**
      * Refresh the message in the remove user warning dialog.
      * @param {string} profilePath The filepath of the URL (must be verified).
      * @param {string} message The message to be written.
      * @param {number|string=} count The number or string to replace $1 in

[dzhioev (left Google), 2016/02/19 06:09:41]

Add a comment about |is_synced_user|.

[Moe, 2016/02/19 15:07:41]

Removed the parameter altogether.

      * |message|. Can be omitted if $1 is not present in |message|.
      */
     updateRemoveWarningDialogSetMessage_: function(profilePath, message,
-                                                   count) {
+                                                   is_synced_user, count) {
       if (profilePath !== this.user.profilePath)
         return;
       // Add localized messages where $1 will be replaced with
       // <span class="total-count"></span>.
+      // and $2 will be replaced with <span class="email"></span>.
       var element = this.querySelector('.action-box-remove-user-warning-text');
       element.textContent = '';

[dzhioev (left Google), 2016/02/19 06:09:41]

Wow, I haven't seen this function before. I think it's weird.
Can we make it a little bit simpler? Something like:

  message = message.replace('$1', '<span class="total-count">' + count +
'</span>');
  message = message.replace('$2', '<span class="email">' +
this.user.emailAddress + '</span>');
  element.innerHTML = message;
          
Also I beleive |is_synced_user| param is excessive, because a) it equals to
|!!this.user.emailAddress| b) the |message| doesn't contain '$2' when
|is_synced_user| is false.

[Moe, 2016/02/19 15:07:41]

Agreed! Done.

 
-      messageParts = message.split('$1');

[Moe, 2016/04/08 13:44:38]

Here's how the logic will more or less look like without setting innerHTML.

+      messageParts = message.split(/(\$1|\$2)/);
       var numParts = messageParts.length;
       for (var j = 0; j < numParts; j++) {
-        element.appendChild(document.createTextNode(messageParts[j]));
-        if (j < numParts - 1) {
-          var elementToAdd = document.createElement('span');
+        var elementToAdd;
+        if (messageParts[j] === '$1') {
+          elementToAdd = document.createElement('span');
           elementToAdd.classList.add('total-count');
           elementToAdd.textContent = count;
           element.appendChild(elementToAdd);
+        } else if (messageParts[j] === '$2' && is_synced_user) {
+          elementToAdd = document.createElement('span');
+          elementToAdd.classList.add('email');
+          elementToAdd.textContent = this.user.emailAddress;
+          element.appendChild(elementToAdd);
+        } else {
+          element.appendChild(document.createTextNode(messageParts[j]));
         }
       }
       this.moveActionMenuUpIfNeeded_();
     },
 
+    substituteElement_: function(placeholder, elementToAdd) {

[dzhioev (left Google), 2016/02/19 06:09:41]

This seems unused.

[Moe, 2016/02/19 15:07:41]

Done.

+
+    },
+
     /**
      * Handles a click event on remove user confirmation button.
      * @param {Event} e Click event.
      */
     handleRemoveUserConfirmationClick_: function(e) {
       if (this.isActionBoxMenuActive) {
         this.isActionBoxMenuActive = false;
         this.removeUser(this.user);
         e.stopPropagation();
       }
@@ skipping 728 matching lines @@
         focus: [this.handleFocus_.bind(this), true /* useCapture */],
         click: [this.handleClick_.bind(this), true],
         mousemove: [this.handleMouseMove_.bind(this), false],
         keydown: [this.handleKeyDown.bind(this), false]
       };
 
       var isDesktopUserManager = Oobe.getInstance().displayType ==
           DISPLAY_TYPE.DESKTOP_USER_MANAGER;
       this.userPodHeight_ = isDesktopUserManager ? DESKTOP_POD_HEIGHT :
                                                    CROS_POD_HEIGHT;
-      // Same for Chrome OS and desktop.
-      this.userPodWidth_ = POD_WIDTH;
+      this.userPodWidth_ = isDesktopUserManager ? DESKTOP_POD_WIDTH :
+                                                   CROS_POD_WIDTH;
     },
 
     /**
      * Returns all the pods in this pod row.
      * @type {NodeList}
      */
     get pods() {
       return Array.prototype.slice.call(this.children);
     },
 
@@ skipping 446 matching lines @@
       return 2 * rowPadding + rows * this.userPodHeight_;
     },
 
     /**
      * Calculates number of columns and rows that podrow should have in order to
      * hold as much its pods as possible for current screen size. Also it tries
      * to choose layout that looks good.
      * @return {{columns: number, rows: number}}
      */
     calculateLayout_: function() {
+      var isDesktopUserManager = Oobe.getInstance().displayType ==
+          DISPLAY_TYPE.DESKTOP_USER_MANAGER;
       var preferredColumns = this.pods.length < COLUMNS.length ?
           COLUMNS[this.pods.length] : COLUMNS[COLUMNS.length - 1];
       var maxWidth = Oobe.getInstance().clientAreaSize.width;
       var columns = preferredColumns;
       while (maxWidth < this.columnsToWidth_(columns) && columns > 1)
         --columns;
       var rows = Math.floor((this.pods.length - 1) / columns) + 1;
       if (getComputedStyle(
           $('signin-banner'), null).getPropertyValue('display') != 'none') {
         rows = Math.min(rows, MAX_NUMBER_OF_ROWS_UNDER_SIGNIN_BANNER);
       }
-      var maxHeigth = Oobe.getInstance().clientAreaSize.height;
-      while (maxHeigth < this.rowsToHeight_(rows) && rows > 1)
-        --rows;
+      if (!isDesktopUserManager) {
+        var maxHeigth = Oobe.getInstance().clientAreaSize.height;
+        while (maxHeigth < this.rowsToHeight_(rows) && rows > 1)
+         --rows;
+      }
       // One more iteration if it's not enough cells to place all pods.
       while (maxWidth >= this.columnsToWidth_(columns + 1) &&
              columns * rows < this.pods.length &&
              columns < MAX_NUMBER_OF_COLUMNS) {
          ++columns;
       }
       return {columns: columns, rows: rows};
     },
 
     /**
@@ skipping 511 matching lines @@
        if (pod && pod.multiProfilesPolicyApplied) {
          pod.userTypeBubbleElement.classList.remove('bubble-shown');
        }
      }
   };
 
   return {
     PodRow: PodRow
   };
 });