jvelociter (SVN) wrote:
> Author: jvelociter
> Date: 2008-07-22 17:57:57 +0200 (Tue, 22 Jul 2008)
> New Revision: 11418
> 
> Modified:
>    
> xwiki-products/xwiki-workspaces/trunk/wiki/src/main/resources/XWSCode/CreateNewWorkspaceForm
> Log:
> XWS-127 Client side validation on workspace creation form does not work under 
> IE6
> 
> 
> Modified: 
> xwiki-products/xwiki-workspaces/trunk/wiki/src/main/resources/XWSCode/CreateNewWorkspaceForm
> ===================================================================
> --- 
> xwiki-products/xwiki-workspaces/trunk/wiki/src/main/resources/XWSCode/CreateNewWorkspaceForm
>       2008-07-22 15:50:14 UTC (rev 11417)
> +++ 
> xwiki-products/xwiki-workspaces/trunk/wiki/src/main/resources/XWSCode/CreateNewWorkspaceForm
>       2008-07-22 15:57:57 UTC (rev 11418)
> @@ -12,9 +12,9 @@
>  <customClass></customClass>
>  <contentAuthor>XWiki.Admin</contentAuthor>
>  <creationDate>1208697558000</creationDate>
> -<date>1214058248000</date>
> -<contentUpdateDate>1214058248000</contentUpdateDate>
> -<version>3.1</version>
> +<date>1216745595000</date>
> +<contentUpdateDate>1216745595000</contentUpdateDate>
> +<version>5.1</version>
>  <title></title>
>  <template></template>
>  <defaultTemplate></defaultTemplate>
> @@ -140,7 +140,29 @@
>           // inform how much chars can be still typed
>           $('spacetitlecharsleft').innerHTML = maxTitleChars - 
> dTitle.value.length;
>       }
> -   }</code>
> +   }
> +
> +/*
> + * getElementsByClassName rewrite.
> + * Developed by Robert Nyman, http://www.robertnyman.com
> + * Code/licensing: http://code.google.com/p/getelementsbyclassname/
> + */
> +function getElementsByClassName(className, tag, elm){
> +     var testClass = new RegExp("(^|\\s)" + className + "(\\s|$)");
> +     var tag = tag || "*";
> +     var elm = elm || document;
> +     var elements = (tag == "*" &amp;&amp; elm.all)? elm.all : 
> elm.getElementsByTagName(tag);
> +     var returnElements = [];
> +     var current;
> +     var length = elements.length;
> +     for(var i=0; i&lt;length; i++){
> +             current = elements[i];
> +             if(testClass.test(current.className)){
> +                     returnElements.push(current);
> +             }
> +     }
> +     return returnElements;
> +}</code>

getElementsByClassName also exists in prototype. We should use that one
instead.

>  </property>
>  <property>
>  <name>Create new workspace validation</name>
> @@ -235,8 +257,8 @@
>     var sDesc = $('XWiki.SpaceClass_0_description');
>     var wAccess = $('XWiki.WorkspaceSpaceClass_0_accesslevel');
>     var wColor = $('XWiki.WorkspaceSpaceClass_0_color');
> -   var colorPreview = 
> wColor.parentNode.parentNode.getElementsByClassName("form-entry-valid")[0];
> -   var 
> vSpan=dTitle.parentNode.parentNode.getElementsByClassName("form-entry-valid")[0];
> +   var colorPreview = 
> getElementsByClassName("form-entry-valid","*",wColor.parentNode.parentNode)[0];
> +   var vSpan = 
> getElementsByClassName("form-entry-valid","*",dTitle.parentNode.parentNode)[0];
>     Event.observe(dTitle,"keyup", titleKeyTyped());   
>     Event.observe(dTitle,"blur", validateTitle());
>     // Green the optional/already filled fields on blur...


-- 
Sergiu Dumitriu
http://purl.org/net/sergiu/

_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to