From 59216e083ceae53c0f7862dcca4af20738b5d2cb Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@fakecake.org>
Date: Sat, 20 Nov 2010 14:53:52 +0300
Subject: [PATCH] use dijit.form.Buttons in refactored filter/label dialogs

---
 functions.js             | 1 +
 modules/pref-filters.php | 8 ++++----
 modules/pref-labels.php  | 6 +++---
 prefs.js                 | 5 +++--
 4 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/functions.js b/functions.js
index bbfd32e4..e9a1f70b 100644
--- a/functions.js
+++ b/functions.js
@@ -1189,6 +1189,7 @@ function quickAddFilter() {
 						parameters: dojo.objectToQuery(this.attr('value')),
 						onComplete: function(transport) {
 							this.hide();
+							notify_progress("Savind data...", true);
 							notify_info(transport.responseText);
 							if (inPreferences()) {
 								updateFilterList();				
diff --git a/modules/pref-filters.php b/modules/pref-filters.php
index 28f30fa6..527df08a 100644
--- a/modules/pref-filters.php
+++ b/modules/pref-filters.php
@@ -157,7 +157,7 @@
 					 name=\"reg_exp\" style=\"font-size : 16px;\" value=\"$reg_exp\">";
 
 			print "<span id=\"filterDlg_dateChkBox\" $date_ops_invisible>";			
-			print "&nbsp;<button onclick=\"return filterDlgCheckDate()\">".
+			print "&nbsp;<button dojoType=\"dijit.form.Button\" onclick=\"return filterDlgCheckDate()\">".
 				__('Check it')."</button>";
 			print "</span>";
 
@@ -244,14 +244,14 @@
 			$reg_exp = htmlspecialchars($reg_exp, ENT_QUOTES); // second escaping seems to be needed for javascript
 
 			print "<div style=\"float : left\">";
-			print "<button onclick='return removeFilter($filter_id, \"$reg_exp\")'>".
+			print "<button dojoType=\"dijit.form.Button\" onclick='return removeFilter($filter_id, \"$reg_exp\")'>".
 				__('Remove')."</button>";
 			print "</div>";
 
-			print "<button onclick=\"return dijit.byId('filterEditDlg').execute()\">".
+			print "<button dojoType=\"dijit.form.Button\" onclick=\"return dijit.byId('filterEditDlg').execute()\">".
 				__('Save')."</button> ";
 
-			print "<button onclick=\"return dijit.byId('filterEditDlg').hide()\">".
+			print "<button dojoType=\"dijit.form.Button\" onclick=\"return dijit.byId('filterEditDlg').hide()\">".
 				__('Cancel')."</button>";
 
 			print "</div>";
diff --git a/modules/pref-labels.php b/modules/pref-labels.php
index 5e025385..e381d72f 100644
--- a/modules/pref-labels.php
+++ b/modules/pref-labels.php
@@ -25,7 +25,7 @@
 			$fg_color = $line['fg_color'];
 			$bg_color = $line['bg_color'];
 
-			print "<span class=\"labelColorIndicator\" id=\"label-editor-indicator\" style='color : $fg_color; background-color : $bg_color'>&alpha;</span>";
+			print "<span class=\"labelColorIndicator\" id=\"label-editor-indicator\" style='color : $fg_color; background-color : $bg_color; margin-bottom : 4px; margin-right : 4px'>&alpha;</span>";
 
 			print "<input style=\"font-size : 16px\" name=\"caption\" 
 				dojoType=\"dijit.form.ValidationTextBox\"
@@ -74,9 +74,9 @@
 #			print "</form>";
 
 			print "<div class=\"dlgButtons\">";
-			print "<button onclick=\"dijit.byId('labelEditDlg').execute()\">".
+			print "<button dojoType=\"dijit.form.Button\" onclick=\"dijit.byId('labelEditDlg').execute()\">".
 				__('Save')."</button>";
-			print "<button onclick=\"dijit.byId('labelEditDlg').hide()\">".
+			print "<button dojoType=\"dijit.form.Button\" onclick=\"dijit.byId('labelEditDlg').hide()\">".
 				__('Cancel')."</button>";
 			print "</div>";
 
diff --git a/prefs.js b/prefs.js
index fa4804ff..fdcf28e8 100644
--- a/prefs.js
+++ b/prefs.js
@@ -302,6 +302,7 @@ function editFilter(id, event) {
 			execute: function() {
 				if (this.validate()) {
 					this.hide();
+					notify_progress("Savind data...", true);
 					new Ajax.Request("backend.php", {
 						parameters: dojo.objectToQuery(this.attr('value')),
 						onComplete: function(transport) {
@@ -1652,8 +1653,8 @@ function removeFilter(id, title) {
 		var ok = confirm(msg);
 	
 		if (ok) {
-			closeInfoBox();
-	
+			dijit.byId("filterEditDlg").hide();
+
 			notify_progress("Removing filter...");
 		
 			var query = "?op=pref-filters&subop=remove&ids="+
-- 
2.39.2