From bd729aa46b24dcb63d2593c1c63bca6b8a1c5ba1 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Thu, 10 Nov 2016 05:18:44 +0300
Subject: [PATCH] cleaned up the button controls...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 ui (gen4)/features/app.js        | 16 +---------
 ui (gen4)/features/ui-widgets.js | 50 ++++++++++++++++++++------------
 2 files changed, 32 insertions(+), 34 deletions(-)

diff --git a/ui (gen4)/features/app.js b/ui (gen4)/features/app.js
index 3f5248c5..598a3855 100755
--- a/ui (gen4)/features/app.js	
+++ b/ui (gen4)/features/app.js	
@@ -261,21 +261,7 @@ var FullScreenControllsActions = actions.Actions({
 	},
 
 	toggleFullScreenControls: ['Interface/',
-		toggler.Toggler(null,
-			function(){ 
-				return this.ribbons.viewer.find('.fullscreen-controls').length > 0 ? 'on' : 'off' },
-			['off', 'on'],
-			function(state){
-				if(state == 'on'){
-					var config = this.config['fullscreen-controls']
-
-					config
-						&& widgets.makeButtonControls(this, 'fullscreen-controls', config)
-
-				} else {
-					this.ribbons.viewer.find('.fullscreen-controls').remove()
-				}
-			})],
+		widgets.makeButtonControlsToggler('fullscreen-controls')],
 })
 
 var FullScreenControlls = 
diff --git a/ui (gen4)/features/ui-widgets.js b/ui (gen4)/features/ui-widgets.js
index 62ad04f0..c7e8ee9e 100755
--- a/ui (gen4)/features/ui-widgets.js	
+++ b/ui (gen4)/features/ui-widgets.js	
@@ -42,12 +42,14 @@ var browseWalk = require('lib/widget/browse-walk')
 var makeButtonControls =
 module.makeButtonControls =
 function(context, cls, data){
+	cls = cls instanceof Array ? cls : cls.split(/\s+/g)
+
 	// remove old versions...
-	context.ribbons.viewer.find('.'+cls).remove()
+	context.ribbons.viewer.find('.'+ cls.join('.')).remove()
 
 	// make container...
 	var controls = $('<div>')
-		.addClass('buttons '+ cls)
+		.addClass('buttons '+ cls.join('.'))
 		.on('mouseover', function(){
 			var t = $(event.target)
 
@@ -90,6 +92,30 @@ function(context, cls, data){
 		.appendTo(context.ribbons.viewer)
 }
 
+var makeButtonControlsToggler =
+module.makeButtonControlsToggler =
+function(cls, cfg){
+	cls = cls instanceof Array ? cls : cls.split(/\s+/g)
+	cfg = cfg || cls[0]
+
+	return toggler.Toggler(null,
+		function(){ 
+			return this.ribbons.viewer.find('.'+ cls.join('.')).length > 0 ? 'on' : 'off' },
+		['off', 'on'],
+		function(state){
+			if(state == 'on'){
+				var config = this.config[cfg]
+
+				config 
+					&& makeButtonControls(this, cls, config)
+
+			} else {
+				this.ribbons.viewer.find('.'+ cls.join('.')).remove()
+			}
+		})
+}
+
+
 
 // XXX make the selector more accurate...
 // 		...at this point this will select the first elem with text which
@@ -1082,25 +1108,11 @@ var MainControlsActions = actions.Actions({
 	},
 
 	toggleMainControls: ['Interface/',
-		toggler.Toggler(null,
-			function(){ 
-				return this.ribbons.viewer.find('.main-controls').length > 0 ? 'on' : 'off' },
-			['off', 'on'],
-			function(state){
-				if(state == 'on'){
-					var config = this.config['main-controls']
-
-					config 
-						&& makeButtonControls(this, 'main-controls', config)
-
-				} else {
-					this.ribbons.viewer.find('.main-controls').remove()
-				}
-			})],
+		makeButtonControlsToggler('main-controls')],
 })
 
-var BrowseActions = 
-module.BrowseActions = core.ImageGridFeatures.Feature({
+var MainControls = 
+module.MainControls = core.ImageGridFeatures.Feature({
 	title: '',
 	doc: '',