You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by id...@apache.org on 2007/11/29 19:21:40 UTC

svn commit: r599532 - in /myfaces/tobago/trunk/theme: scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/

Author: idus
Date: Thu Nov 29 10:21:35 2007
New Revision: 599532

URL: http://svn.apache.org/viewvc?rev=599532&view=rev
Log:
TOBAGO-561: ensure that correct default button is active after Ajax request

Modified:
    myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ButtonRenderer.java
    myfaces/tobago/trunk/theme/standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js

Modified: myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ButtonRenderer.java
URL: http://svn.apache.org/viewvc/myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ButtonRenderer.java?rev=599532&r1=599531&r2=599532&view=diff
==============================================================================
--- myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ButtonRenderer.java (original)
+++ myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ButtonRenderer.java Thu Nov 29 10:21:35 2007
@@ -54,8 +54,6 @@
       LOG.error("Wrong type: Need " + UICommand.class.getName() + ", but was " + component.getClass().getName());
       return;
     }
-
-
   }
 
   public void encodeEnd(FacesContext facesContext,
@@ -95,7 +93,7 @@
 
     String imageName = (String) command.getAttributes().get(ATTR_IMAGE);
     if (imageName != null) {
-      String image = null;
+      String image;
       if (imageName.startsWith("HTTP:") || imageName.startsWith("FTP:")
           || imageName.startsWith("/")) {
         image = imageName;
@@ -130,6 +128,9 @@
     if (ComponentUtil.getBooleanAttribute(component, ATTR_DEFAULT_COMMAND)) {
       boolean transition = ComponentUtil.getBooleanAttribute(command, ATTR_TRANSITION);
       HtmlRendererUtil.setDefaultTransition(facesContext, transition);
+
+      HtmlRendererUtil.writeScriptLoader(facesContext, null, new String[]{
+          "Tobago.setDefaultAction('" + command.getClientId(facesContext) + "')"});      
     }
   }
 

Modified: myfaces/tobago/trunk/theme/standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js
URL: http://svn.apache.org/viewvc/myfaces/tobago/trunk/theme/standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js?rev=599532&r1=599531&r2=599532&view=diff
==============================================================================
--- myfaces/tobago/trunk/theme/standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js (original)
+++ myfaces/tobago/trunk/theme/standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js Thu Nov 29 10:21:35 2007
@@ -1483,8 +1483,14 @@
       }
     }
     return this.browser;
-  }
+  },
 
+  setDefaultAction: function(defaultActionId) {
+    var field = Tobago.element(Tobago.page.id + Tobago.SUB_COMPONENT_SEP + "form-action");
+    if (field) {
+      field.value = defaultActionId;
+    }
+  }
 };
 
 



Re: svn commit: r599532 - in /myfaces/tobago/trunk/theme: scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/

Posted by Volker Weber <v....@inexso.de>.
Hi Arvid,

setDefaultAction: function(defaultActionId) {
  var field = Tobago.element(Tobago.page.id + Tobago.SUB_COMPONENT_SEP
+ "form-action");
  if (field) {
    field.value = defaultActionId;
  }
}

could be simplified to

setDefaultAction: function(defaultActionId) {
  Tobago.action.value = defaultActionId;
}


Regards,
    Volker



2007/11/29, idus@apache.org <id...@apache.org>:
> Author: idus
> Date: Thu Nov 29 10:21:35 2007
> New Revision: 599532
>
> URL: http://svn.apache.org/viewvc?rev=599532&view=rev
> Log:
> TOBAGO-561: ensure that correct default button is active after Ajax request
>
> Modified:
>     myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ButtonRenderer.java
>     myfaces/tobago/trunk/theme/standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js
>
> Modified: myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ButtonRenderer.java
> URL: http://svn.apache.org/viewvc/myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ButtonRenderer.java?rev=599532&r1=599531&r2=599532&view=diff
> ==============================================================================
> --- myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ButtonRenderer.java (original)
> +++ myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/ButtonRenderer.java Thu Nov 29 10:21:35 2007
> @@ -54,8 +54,6 @@
>        LOG.error("Wrong type: Need " + UICommand.class.getName() + ", but was " + component.getClass().getName());
>        return;
>      }
> -
> -
>    }
>
>    public void encodeEnd(FacesContext facesContext,
> @@ -95,7 +93,7 @@
>
>      String imageName = (String) command.getAttributes().get(ATTR_IMAGE);
>      if (imageName != null) {
> -      String image = null;
> +      String image;
>        if (imageName.startsWith("HTTP:") || imageName.startsWith("FTP:")
>            || imageName.startsWith("/")) {
>          image = imageName;
> @@ -130,6 +128,9 @@
>      if (ComponentUtil.getBooleanAttribute(component, ATTR_DEFAULT_COMMAND)) {
>        boolean transition = ComponentUtil.getBooleanAttribute(command, ATTR_TRANSITION);
>        HtmlRendererUtil.setDefaultTransition(facesContext, transition);
> +
> +      HtmlRendererUtil.writeScriptLoader(facesContext, null, new String[]{
> +          "Tobago.setDefaultAction('" + command.getClientId(facesContext) + "')"});
>      }
>    }
>
>
> Modified: myfaces/tobago/trunk/theme/standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js
> URL: http://svn.apache.org/viewvc/myfaces/tobago/trunk/theme/standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js?rev=599532&r1=599531&r2=599532&view=diff
> ==============================================================================
> --- myfaces/tobago/trunk/theme/standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js (original)
> +++ myfaces/tobago/trunk/theme/standard/src/main/resources/org/apache/myfaces/tobago/renderkit/html/standard/standard/script/tobago.js Thu Nov 29 10:21:35 2007
> @@ -1483,8 +1483,14 @@
>        }
>      }
>      return this.browser;
> -  }
> +  },
>
> +  setDefaultAction: function(defaultActionId) {
> +    var field = Tobago.element(Tobago.page.id + Tobago.SUB_COMPONENT_SEP + "form-action");
> +    if (field) {
> +      field.value = defaultActionId;
> +    }
> +  }
>  };
>
>
>
>
>