[Yanel-commits] rev 47310 - public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client

michi at wyona.com michi at wyona.com
Thu Jan 28 11:54:49 CET 2010


Author: michi
Date: 2010-01-28 11:54:48 +0100 (Thu, 28 Jan 2010)
New Revision: 47310

Modified:
   public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java
Log:
javadoc added, arrows order changed and TODO added

Modified: public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java
===================================================================
--- public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java	2010-01-28 10:53:40 UTC (rev 47309)
+++ public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AddRemoveIdentitiesWidget.java	2010-01-28 10:54:48 UTC (rev 47310)
@@ -29,7 +29,7 @@
 import com.google.gwt.user.client.ui.Widget;
 
 /**
- *
+ * Move users/groups from identities list into policy and vice versa
  */
 public class AddRemoveIdentitiesWidget extends Composite implements ClickListener {
 
@@ -48,20 +48,22 @@
     public AddRemoveIdentitiesWidget(ListBox identitiesListBox, ListBox policyListBox, PolicyListBoxWidget policyLBW) {
         initWidget(fp);
 
-        removeButton = new Button("<", this);
-        fp.add(removeButton);
         addButton = new Button(">", this);
         fp.add(addButton);
 
+        removeButton = new Button("<", this);
+        fp.add(removeButton);
+
         this.identitiesLB = identitiesListBox;
         this.policyLB = policyListBox;
         this.policyLBW = policyLBW;
     }
 
     /**
-     *
+     * Move item from one list to the other
      */
     public void onClick(Widget sender) {
+        // TODO: Allow multiple selection
         if (sender == addButton) {
             int i = identitiesLB.getSelectedIndex();
             if (i >= 0) {



More information about the Yanel-commits mailing list