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

[simon at wyona.com]

Author: simon
Date: 2011-04-05 13:27:47 +0200 (Tue, 05 Apr 2011)
New Revision: 57690

Modified:
   public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AccessPolicyEditor.java
   public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/IdentitiesListBoxWidget.java
   public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/PolicyListBoxWidget.java
Log:
added more styles and containers (panels) to make it easier to style the editor.

Modified: public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AccessPolicyEditor.java
===================================================================
--- public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AccessPolicyEditor.java	2011-04-05 11:08:37 UTC (rev 57689)
+++ public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/AccessPolicyEditor.java	2011-04-05 11:27:47 UTC (rev 57690)
@@ -159,7 +159,7 @@
                 }
             }
         });
-        saveButton.setStyleName("gwt-wyona-SaveButton");
+        saveButton.addStyleName("gwt-wyona-SaveButton");
         buttonHP.add(saveButton);
 
         // Cancel Button
@@ -178,7 +178,7 @@
                 $wnd.location.href=url;
             }-*/; 
         });
-        saveButton.setStyleName("gwt-wyona-CancelButton");
+        cancelButton.addStyleName("gwt-wyona-CancelButton");
         buttonHP.add(cancelButton);
 
         //vp.add(new Label("Wyona Access Control Policy (GWT) Editor version 1.0-dev-r47029"));

Modified: public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/IdentitiesListBoxWidget.java
===================================================================
--- public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/IdentitiesListBoxWidget.java	2011-04-05 11:08:37 UTC (rev 57689)
+++ public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/IdentitiesListBoxWidget.java	2011-04-05 11:27:47 UTC (rev 57690)
@@ -54,10 +54,16 @@
         this.language = language;
         initWidget(vp);
 
-        vp.add(new Label(I18n.getLabel("list-box-identities", language) + ":"));
+        VerticalPanel titlePanel = new VerticalPanel();
+        titlePanel.setStyleName("gwt-wyona-identity-list-box-title");
+        
+//        vp.add(new Label(I18n.getLabel("list-box-identities", language) + ":"));
+        titlePanel.add(new Label(I18n.getLabel("list-box-identities", language) + ":"));
+        vp.add(titlePanel);
 
         boolean isMultipleSelect = true;
         lb = new ListBox(isMultipleSelect); // NOTE: ListBox#setMultipleSelect(true) can spuriously fail on IE 6.0
+        lb.setStyleName("gwt-wyona-identity-list-box");
         lb.addClickListener(this);
 
         lb.clear();

Modified: public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/PolicyListBoxWidget.java
===================================================================
--- public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/PolicyListBoxWidget.java	2011-04-05 11:08:37 UTC (rev 57689)
+++ public/yanel/contributions/ajax-components/src/access-policy-editor/java/org/wyona/security/gwt/accesspolicyeditor/client/PolicyListBoxWidget.java	2011-04-05 11:27:47 UTC (rev 57690)
@@ -52,13 +52,22 @@
     public PolicyListBoxWidget(int visibleItemCount, User[] users, Group[] groups, boolean useInheritedPolicies, String language) {
         initWidget(vp);
 
-        vp.add(new Label("Policy:"));
+        VerticalPanel titlePanel = new VerticalPanel();
+        titlePanel.setStyleName("gwt-wyona-policy-list-box-title");
+        
+        titlePanel.add(new Label("Policy:"));
+        
+//        vp.add();
 
         policyInheritanceCB = new CheckBox(I18n.getLabel("inherit-rights-label", language));
         setInheritRightsFlag(useInheritedPolicies);
-        vp.add(policyInheritanceCB);
+//        vp.add(policyInheritanceCB);
+        titlePanel.add(policyInheritanceCB);
+        
+        vp.add(titlePanel);
 
         lb = new ListBox(true); // NOTE: ListBox#setMultipleSelect(true) can spuriously fail on IE 6.0
+        lb.setStyleName("gwt-wyona-policy-list-box");
         lb.addClickListener(this);
         //setIdentities(visibleItemCount, users, groups);
         displayLoading(visibleItemCount);