refactoring and tweaking...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>

ui/css/editor.css

@@ -48,8 +48,8 @@
 	min-height: 15px;
 }
 .sub-panel,
-.panel button,
-.panel .state {
+.sub-panel button,
+.sub-panel .state {
 	margin: 1px;
 	font-size: 11px;
 	border: solid 1px #aaa;
@@ -71,12 +71,15 @@
 	*/
 }
 .sub-panel .sub-panel-content {
+	margin: 10px;
+	/*
 	margin-left: 10px;
 	margin-right: 10px;
+	*/
 }
 
-.panel button:active,
-.panel .state:active {
+.sub-panel button:active,
+.sub-panel .state:active {
 	background: silver;
 }
 

ui/experiments/panels.html

@@ -62,6 +62,10 @@ $(function(){
 
 	makeSubPanel('Test Sub Panel C', $('<h3>Panel C</h3>'), panel, false)
 
+	makeFilterPanel(panel, '.moo')
+
+	makeSnapshotsPanel(panel, '.moo')
+
 
 	$('body')
 		.append(panel)	

ui/lib/editor.js

@@ -246,9 +246,14 @@ function makeLogRange(text, filter, target){
 }
 
 
-function makeFilterPanel(parent, target){
-	return makeSubPanel('Edit: Filters', true, parent)
-		.find('.sub-panel-content')
+
+/**********************************************************************
+* Panels...
+*/
+
+function makeFilterPanel(parent, target, open){
+	return makeSubPanel('Edit: Filters', 
+		$('<div>')
 			.append($('<div class="filter-list"/>')
 				//.append(makeLogRange('Gamma:', 'gamma', target))
 				.append(makeLogRange('Brightness:', 'brightness', target))
@@ -268,7 +273,7 @@ function makeFilterPanel(parent, target){
 					img.css('-webkit-filter', sortFilterStr(img.css('-webkit-filter')))
 				}))
 			.append($('<hr>'))
-			.append('Reset: ')
+			.append('<span>Reset: <span>')
 			.append($('<button>Values</button>')
 				.click(function(){
 					$('.reset').click()
@@ -282,11 +287,13 @@ function makeFilterPanel(parent, target){
 					$('.reset').click()
 					sortFilterSliders(DEFAULT_FILTER_ORDER)
 				}))
+			.children(),
+		parent, open)
 }
 
-function makeSnapshotsPanel(parent, target){
-	return makeSubPanel('Edit: Snapshots', true, parent)
-		.find('.sub-panel-content')
+function makeSnapshotsPanel(parent, target, open){
+	return makeSubPanel('Edit: Snapshots', 
+		$('<div>')
 			.append($('<div class="states"/>'))
 			.append($('<hr>'))
 			.append($('<button/>')
@@ -309,6 +316,8 @@ function makeSnapshotsPanel(parent, target){
 					}
 					
 				}))
+			.children(),
+		parent, open)
 }
 
 

ui/lib/panels.js

@@ -248,9 +248,10 @@ function makeSidePanel(side, autohide){
 }
 
 
-//function makeSubPanel(title, open, parent, content_resizable){
+// NOTE: if parent is not given this will create a new panel...
 function makeSubPanel(title, content, parent, open, content_resizable){
 	title = title == null || title.trim() == '' ? ' ' : title
+	parent = parent == null ? makePanel() : parent
 
 	open = open == null ? true : open
 	content_resizable = content_resizable == null