Merge branch 'master' of https://github.com/designcreateplay/NodeBB

public/src/admin.js

@@ -0,0 +1,23 @@
+"use strict";
+
+var admin = {};
+
+(function() {
+
+	admin.enableColorPicker = function(inputEl, callback) {
+		(inputEl instanceof jQuery ? inputEl : $(inputEl)).each(function() {
+			var $this = $(this);
+
+			$this.ColorPicker({
+				color: $this.val() || '#000',
+				onChange: function(hsb, hex) {
+					$this.val('#' + hex);
+					if (typeof callback === 'function') {
+						callback(hsb, hex);
+					}
+				}
+			});
+		});		
+	};
+
+}());

public/src/forum/admin/categories.js

@@ -139,21 +139,17 @@ define(['uploader'], function(uploader) {
 		}
 
 		function enableColorPicker(idx, inputEl) {
-			var	$inputEl = $(inputEl),
+			var $inputEl = $(inputEl),
 				previewEl = $inputEl.parents('[data-cid]').find('.preview-box');
 
-			$inputEl.ColorPicker({
+			admin.enableColorPicker($inputEl, function(hsb, hex) {
-				color: $inputEl.val() || '#000',
+				if ($inputEl.attr('data-name') === 'bgColor') {
-				onChange: function(hsb, hex) {
+					previewEl.css('background', '#' + hex);
-					$inputEl.val('#' + hex);
+				} else if ($inputEl.attr('data-name') === 'color') {
-					if ($inputEl.attr('data-name') === 'bgColor') {
+					previewEl.css('color', '#' + hex);
-						previewEl.css('background', '#' + hex);
-					} else if ($inputEl.attr('data-name') === 'color') {
-						previewEl.css('color', '#' + hex);
-					}
-
-					modified($inputEl[0]);
 				}
+
+				modified($inputEl[0]);
 			});
 		}
 

public/src/forum/admin/themes.js

@@ -120,6 +120,8 @@ define(['forum/admin/settings'], function(Settings) {
 
 		Themes.prepareWidgets();
 
+		populateBranding();
+		admin.enableColorPicker($('.branding'));
 		Settings.prepare();
 	};
 
@@ -320,19 +322,21 @@ define(['forum/admin/settings'], function(Settings) {
 		});
 	};
 
-	require(['settings'], function (settings) {
+	function populateBranding() {
-		var wrapper = $('#branding');
+		require(['settings'], function (settings) {
+			var wrapper = $('#branding');
 
-		settings.sync('branding', wrapper);
+			settings.sync('branding', wrapper);
 
-		$('#save-branding').click(function(event) {
+			$('#save-branding').click(function(event) {
-			settings.persist('branding', $('#branding'), function() {
+				settings.persist('branding', $('#branding'), function() {
-				socket.emit('admin.themes.updateBranding');
+					socket.emit('admin.themes.updateBranding');
-			});
+				});
 
-			event.preventDefault();
+				event.preventDefault();
+			});
 		});
-	});
+	}
 
 	return Themes;
 });

src/meta.js

@@ -155,15 +155,14 @@ var fs = require('fs'),
 						db.setObject('config', themeData, next);
 					}
 				], callback);
+
+				Meta.restartRequired = true;
 				break;
 
 			case 'bootswatch':
-				db.setObjectField('config', 'theme:src', data.src, callback);
+				Meta.configs.set('theme:src', data.src, callback);
 				break;
 			}
-
-			// Restart Required flag
-			Meta.restartRequired = true;
 		}
 	};