From 6248bad0f7b23c16049045ea150a11948b0ebeb1 Mon Sep 17 00:00:00 2001
From: Roeland Jago Douma <rullzer@owncloud.com>
Date: Thu, 17 Dec 2015 07:56:02 +0100
Subject: [PATCH] Add a default size to the avatar placeholders

This removed the need to do an avatar request on the "empty" row in the
user settings.
---
 core/js/avatar.js          | 4 ----
 core/js/placeholder.js     | 8 ++++++--
 settings/js/users/users.js | 2 +-
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/core/js/avatar.js b/core/js/avatar.js
index 10214c35fb..78b8c2a8cf 100644
--- a/core/js/avatar.js
+++ b/core/js/avatar.js
@@ -2,8 +2,4 @@ $(document).ready(function(){
 	if (OC.currentUser) {
 
 	}
-	// User settings
-	$.each($('td.avatar .avatardiv'), function(i, element) {
-		$(element).avatar($(element).parent().parent().data('uid'), 32);
-	});
 });
diff --git a/core/js/placeholder.js b/core/js/placeholder.js
index 74bb9b1881..26eb3507d7 100644
--- a/core/js/placeholder.js
+++ b/core/js/placeholder.js
@@ -47,16 +47,20 @@
  */
 
 (function ($) {
-	$.fn.imageplaceholder = function(seed, text) {
+	$.fn.imageplaceholder = function(seed, text, size) {
 		// set optional argument "text" to value of "seed" if undefined
 		text = text || seed;
 
 		var hash = md5(seed),
 			maxRange = parseInt('ffffffffffffffffffffffffffffffff', 16),
 			hue = parseInt(hash, 16) / maxRange * 256,
-			height = this.height();
+			height = this.height() || size || 32;
 		this.css('background-color', 'hsl(' + hue + ', 90%, 65%)');
 
+		// Placeholders are square
+		this.height(height);
+		this.width(height);
+
 		// CSS rules
 		this.css('color', '#fff');
 		this.css('font-weight', 'normal');
diff --git a/settings/js/users/users.js b/settings/js/users/users.js
index 9b95408dd0..aea9405357 100644
--- a/settings/js/users/users.js
+++ b/settings/js/users/users.js
@@ -68,7 +68,7 @@ var UserList = {
 			if (user.isAvatarAvailable === true) {
 				$('div.avatardiv', $tr).avatar(user.name, 32, undefined, undefined, undefined, user.displayname);
 			} else {
-				$('div.avatardiv', $tr).imageplaceholder(user.displayname);
+				$('div.avatardiv', $tr).imageplaceholder(user.displayname, undefined, 32);
 			}
 		}