From 5750ebdd831f7f3e2dd5c08031a258ee448c7afa Mon Sep 17 00:00:00 2001
From: Kevin McCarthy <kmccarth@redhat.com>
Date: Tue, 2 Oct 2007 10:49:44 -0700
Subject: Assorted UI fixes: - Change sort functions to be on entities, so can
 use on the view pages too - Fix bug: empty ajax search on useredit blows up -
 Filter illegal characters from suggest uid/email methods - Rename first/last
 name fields - Make default font family sans-serif - Speed up effect
 appear/fade rendering - Add buttons to top and bottom of pages - Make
 grouplist sortable - Add noscript warning to welcome page

---
 ipa-server/ipa-gui/ipagui/static/javascript/dynamicedit.js | 14 +++++++-------
 ipa-server/ipa-gui/ipagui/static/javascript/tablekit.js    |  6 ++++--
 2 files changed, 11 insertions(+), 9 deletions(-)

(limited to 'ipa-server/ipa-gui/ipagui/static/javascript')

diff --git a/ipa-server/ipa-gui/ipagui/static/javascript/dynamicedit.js b/ipa-server/ipa-gui/ipagui/static/javascript/dynamicedit.js
index 05c22998a..b7dd95ea1 100644
--- a/ipa-server/ipa-gui/ipagui/static/javascript/dynamicedit.js
+++ b/ipa-server/ipa-gui/ipagui/static/javascript/dynamicedit.js
@@ -102,7 +102,7 @@ function addmember(dn, info) {
   var undolink = document.createElement('a');
   undolink.setAttribute('href', '');
   undolink.setAttribute('onclick',
-    'new Effect.Fade(Element.up(this), {afterFinish: removeElement});' +
+    'new Effect.Fade(Element.up(this), {afterFinish: removeElement, duration: 0.75});' +
     'added_hash.remove("' + jsStringEscape(dn) + '");' +
     'return false;');
   undolink.appendChild(document.createTextNode("undo"));
@@ -123,8 +123,8 @@ function addmember(dn, info) {
 function addmemberHandler(element, dn, info) {
   var newdiv = addmember(dn, info)
   if (newdiv != null) {
-    new Effect.Fade(Element.up(element));
-    new Effect.Appear(newdiv);
+    new Effect.Fade(Element.up(element), {duration: 0.75});
+    new Effect.Appear(newdiv, {duration: 0.75});
     /* Element.up(element).remove(); */
   }
 }
@@ -139,8 +139,8 @@ function removemember(dn, info) {
   var undolink = document.createElement('a');
   undolink.setAttribute('href', '');
   undolink.setAttribute('onclick',
-    'new Effect.Fade(Element.up(this), {afterFinish: removeElement});' +
-    "new Effect.Appear($('" + orig_div_id + "'));" +
+    'new Effect.Fade(Element.up(this), {afterFinish: removeElement, duration: 0.75});' +
+    "new Effect.Appear($('" + orig_div_id + "'), {duration: 0.75});" +
     'return false;');
   undolink.appendChild(document.createTextNode("undo"));
   newdiv.appendChild(undolink);
@@ -159,8 +159,8 @@ function removemember(dn, info) {
 
 function removememberHandler(element, dn, info) {
   var newdiv = removemember(dn, info);
-  new Effect.Fade(Element.up(element));
-  new Effect.Appear(newdiv);
+  new Effect.Fade(Element.up(element), {duration: 0.75});
+  new Effect.Appear(newdiv, {duration: 0.75});
   /* Element.up(element).remove(); */
 }
 
diff --git a/ipa-server/ipa-gui/ipagui/static/javascript/tablekit.js b/ipa-server/ipa-gui/ipagui/static/javascript/tablekit.js
index 2851f8431..540369489 100644
--- a/ipa-server/ipa-gui/ipagui/static/javascript/tablekit.js
+++ b/ipa-server/ipa-gui/ipagui/static/javascript/tablekit.js
@@ -291,7 +291,9 @@ TableKit.Sortable = {
 		
 		if(cell.hasClassName(op.noSortClass)) {return;}	
 		
-		order = order ? order : (cell.hasClassName(op.descendingClass) ? 1 : -1);
+		// order = order ? order : (cell.hasClassName(op.descendingClass) ? 1 : -1);
+                // kmccarth - change default sort order to ascending
+		order = order ? order : (cell.hasClassName(op.ascendingClass) ? -1 : 1);
 		var rows = TableKit.getBodyRows(table);
 
 		if(cell.hasClassName(op.ascendingClass) || cell.hasClassName(op.descendingClass)) {
@@ -843,4 +845,4 @@ if(window.FastInit) {
 	FastInit.addOnLoad(TableKit.load);
 } else {
 	Event.observe(window, 'load', TableKit.load);
-}
\ No newline at end of file
+}
-- 
cgit