[Freeipa-devel] [PATCH] Move undo button next to selected radio button.

[Endi Sukma Dewata]

Hi,

The attached patch should fix item #2 in this bug:
https://fedorahosted.org/freeipa/ticket/671

Thanks!

--
Endi S. Dewata

From 7a1f7cb1e0ad73316fbeb132a16e09195e6147fd Mon Sep 17 00:00:00 2001
From: Endi S. Dewata <edew...@redhat.com>
Date: Fri, 7 Jan 2011 08:26:20 +0700
Subject: [PATCH] Move undo button next to selected radio button.

---
 install/static/sudorule.js |   55 ++++++++++++++++++++++++++++++++------------
 1 files changed, 40 insertions(+), 15 deletions(-)

diff --git a/install/static/sudorule.js b/install/static/sudorule.js
index ec90bf618d731fc1581ff1de1c9449d36c7e2959..23135c00b8dc2734db976cf06f7e5473aa38ea17 100755
--- a/install/static/sudorule.js
+++ b/install/static/sudorule.js
@@ -596,43 +596,68 @@ function ipa_sudorule_details_command_section(spec){
             title: param_info ? param_info.doc : 'cmdcategory'
         }).appendTo(container);
 
+        var undo = $('<span/>', {
+            'name': 'undo',
+            'class': 'ui-state-highlight ui-corner-all',
+            'style': 'display: none;',
+            'html': 'undo'
+        }).appendTo(span);
+
         $('<input/>', {
-            'type': 'radio',
-            'name': 'cmdcategory',
-            'value': 'allow'
+            type: 'radio',
+            name: 'cmdcategory',
+            value: 'allow',
+            click: function() {
+                undo.detach();
+                undo.appendTo(option1_undo);
+            }
         }).appendTo(span);
 
+        // TODO: replace with i18n label
         span.append('Allow Any Command / Group');
 
         span.append(' ');
 
-        $('<span/>', {
-            'name': 'undo',
-            'class': 'ui-state-highlight ui-corner-all',
-            'style': 'display: none;',
-            'html': 'undo'
-        }).appendTo(span);
+        var option1_undo = $('<span/>').appendTo(span);
 
         span.append('<br/>');
 
         $('<input/>', {
-            'type': 'radio',
-            'name': 'cmdcategory',
-            'value': 'deny'
+            type: 'radio',
+            name: 'cmdcategory',
+            value: 'deny',
+            click: function() {
+                undo.detach();
+                undo.appendTo(option2_undo);
+            }
         }).appendTo(span);
 
+        // TODO: replace with i18n label
         span.append('Deny Any Command / Group');
 
+        span.append(' ');
+
+        var option2_undo = $('<span/>').appendTo(span);
+
         span.append('<br/>');
 
         $('<input/>', {
-            'type': 'radio',
-            'name': 'cmdcategory',
-            'value': ''
+            type: 'radio',
+            name: 'cmdcategory',
+            value: '',
+            click: function() {
+                undo.detach();
+                undo.appendTo(option3_undo);
+            }
         }).appendTo(span);
 
+        // TODO: replace with i18n label
         span.append('Specific Command / Group');
 
+        span.append(' ');
+
+        var option3_undo = $('<span/>').appendTo(span);
+
         // TODO: replace with i18n label
         $('<h3/>', {
             text: 'Allow',
-- 
1.6.6.1


_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel