Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(144)

Unified Diff: ui/login/account_picker/md_user_pod_row.js

Issue 2956873002: Don't hide profile statistics counters with zero results (Closed)
Patch Set: always remove 'has-no-stats' when stats are loaded Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | ui/login/account_picker/user_pod_row.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/login/account_picker/md_user_pod_row.js
diff --git a/ui/login/account_picker/md_user_pod_row.js b/ui/login/account_picker/md_user_pod_row.js
index 0ac5ea25fa1a85bd94214198d475cf7bfdc854c7..0135bbc5946687cc6d310d2e39e1713c714ff8af 100644
--- a/ui/login/account_picker/md_user_pod_row.js
+++ b/ui/login/account_picker/md_user_pod_row.js
@@ -1841,14 +1841,12 @@ cr.define('login', function() {
}
// this.classList is used for selecting the appropriate dialog.
- if (total_count)
- this.classList.remove('has-no-stats');
+ this.classList.remove('has-no-stats');
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');
@@ -3574,7 +3572,7 @@ cr.define('login', function() {
} else {
// When the user count exceeds the limit (currently set to 2), only the
// main pod still has pow row as parent, all other pods should be
- // appended to the container with scroll bar.
+ // appended to the container with scroll bar.
for (var pod of pods) {
if (pod == this.mainPod_)
this.appendChild(pod);
@@ -3672,7 +3670,7 @@ cr.define('login', function() {
var MIDDLE_PADDING = this.isPortraitMode_() ? 84 : 220;
var contentsWidth = LEFT_PADDING +
CROS_POD_WIDTH + MIDDLE_PADDING + CROS_SMALL_POD_WIDTH;
- var blankWidth = this.screenSize.width - contentsWidth;
+ var blankWidth = this.screenSize.width - contentsWidth;
var actualLeftPadding = LEFT_PADDING;
actualLeftPadding +=
this.isPortraitMode_() ? blankWidth * 2 / 3 : blankWidth / 2;
@@ -3704,10 +3702,10 @@ cr.define('login', function() {
return;
}
}
-
+
// Start positioning of the main pod and the smallPodsContainer.
this.mainPod_.left = actualLeftPadding;
- this.mainPod_.top = (this.screenSize.height - CROS_POD_HEIGHT) / 2;
+ this.mainPod_.top = (this.screenSize.height - CROS_POD_HEIGHT) / 2;
this.smallPodsContainer.style.left =
cr.ui.toCssPx(actualLeftPadding + CROS_POD_WIDTH + MIDDLE_PADDING);
this.smallPodsContainer.style.top = cr.ui.toCssPx(0);
@@ -3780,7 +3778,7 @@ cr.define('login', function() {
SCROLL_RIGHT_PADDING);
// SCROLL_TOP_PADDING denotes the smallest top padding we can tolerate
- // before allowing the container to overflow and show the scroll bar.
+ // before allowing the container to overflow and show the scroll bar.
var actualTopPadding = SCROLL_TOP_PADDING;
if ((this.screenSize.height - scrollHeight) / 2 > actualTopPadding) {
// Edge case: the total height of the scrollable container does not
@@ -3794,10 +3792,10 @@ cr.define('login', function() {
// The scroll bar will definitely be shown if we reach here. A gradient
// mask is applied to avoid blocking the header bar if the virtual
// keyboard is not shown. When the keyboard is shown, there's no need
- // to add the mask and the original top padding value should be kept.
+ // to add the mask and the original top padding value should be kept.
actualTopPadding = SCROLL_MASK_HEIGHT;
this.showScrollMask_();
- }
+ }
// Start positioning of the small pods inside the smallPodsContainer.
var topPadding = actualTopPadding;
@@ -4098,7 +4096,7 @@ cr.define('login', function() {
var top = pod.top;
// Edge case: paddingBottom should be switched too because there's a
// chance that the small pod was at the end of the scrollable container
- // and had a non-zero paddingBottom.
+ // and had a non-zero paddingBottom.
var paddingBottom = pod.style.paddingBottom;
var parent = pod.parentNode;
parent.removeChild(pod);
« no previous file with comments | « no previous file | ui/login/account_picker/user_pod_row.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698