You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by dm...@apache.org on 2002/10/03 07:00:47 UTC

cvs commit: jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/taglib/html ELRadioTag.java

dmkarr      2002/10/02 22:00:47

  Modified:    contrib/struts-el/src/share/org/apache/strutsel/taglib/html
                        ELRadioTag.java
  Log:
  Uncommented "onmousemove", "onmouseout", and "onmouseover" attribute handlers,
  as I found the base tag was properly handling these.  The "onmouseup" handler
  is still commented out, as the bug in the base tag isn't fixed yet.
  
  Revision  Changes    Path
  1.4       +24 -23    jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/taglib/html/ELRadioTag.java
  
  Index: ELRadioTag.java
  ===================================================================
  RCS file: /home/cvs/jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/taglib/html/ELRadioTag.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- ELRadioTag.java	1 Oct 2002 04:25:50 -0000	1.3
  +++ ELRadioTag.java	3 Oct 2002 05:00:47 -0000	1.4
  @@ -227,25 +227,26 @@
               setOnmousedown(null);
           }
   
  -//         try {
  -//             setOnmousemove((String) evalAttr("onmousemove", getOnmousemove(), 
  -//                                              String.class));
  -//         } catch (NullAttributeException ex) {
  -//             setOnmousemove(null);
  -//         }
  +        try {
  +            setOnmousemove((String) evalAttr("onmousemove", getOnmousemove(), 
  +                                             String.class));
  +        } catch (NullAttributeException ex) {
  +            setOnmousemove(null);
  +        }
   
  -//         try {
  -//             setOnmouseout((String) evalAttr("onmouseout", getOnmouseout(), String.class));
  -//         } catch (NullAttributeException ex) {
  -//             setOnmouseout(null);
  -//         }
  +        try {
  +            setOnmouseout((String) evalAttr("onmouseout", getOnmouseout(),
  +                                            String.class));
  +        } catch (NullAttributeException ex) {
  +            setOnmouseout(null);
  +        }
   
  -//         try {
  -//             setOnmouseover((String) evalAttr("onmouseover", getOnmouseover(), 
  -//                                              String.class));
  -//         } catch (NullAttributeException ex) {
  -//             setOnmouseover(null);
  -//         }
  +        try {
  +            setOnmouseover((String) evalAttr("onmouseover", getOnmouseover(), 
  +                                             String.class));
  +        } catch (NullAttributeException ex) {
  +            setOnmouseover(null);
  +        }
   
   //         try {
   //             setOnmouseup((String) evalAttr("onmouseup", getOnmouseup(), String.class));
  
  
  

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>


Re: cvs commit: jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/taglib/html ELRadioTag.java

Posted by "David M. Karr" <dm...@earthlink.net>.
>>>>> "Craig" == Craig R McClanahan <cr...@apache.org> writes:

    Craig> On 3 Oct 2002 dmkarr@apache.org wrote:

    >> Date: 3 Oct 2002 05:00:47 -0000
    >> From: dmkarr@apache.org
    >> Reply-To: Struts Developers List <st...@jakarta.apache.org>
    >> To: jakarta-struts-cvs@apache.org
    >> Subject: cvs commit:
    >> jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/taglib/html
    >> ELRadioTag.java
    >> 
    >> dmkarr      2002/10/02 22:00:47
    >> 
    >> Modified:    contrib/struts-el/src/share/org/apache/strutsel/taglib/html
    >> ELRadioTag.java
    >> Log:
    >> Uncommented "onmousemove", "onmouseout", and "onmouseover" attribute handlers,
    >> as I found the base tag was properly handling these.  The "onmouseup" handler
    >> is still commented out, as the bug in the base tag isn't fixed yet.

    Craig> You're more than welcome to fix the bug in the base tag :-).

I thought it was likely I would do this, I just wanted to get the bug logged
first.

-- 
===================================================================
David M. Karr          ; Java/J2EE/XML/Unix/C++
dmkarr@earthlink.net


--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>


Re: cvs commit: jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/taglib/html ELRadioTag.java

Posted by "Craig R. McClanahan" <cr...@apache.org>.

On 3 Oct 2002 dmkarr@apache.org wrote:

> Date: 3 Oct 2002 05:00:47 -0000
> From: dmkarr@apache.org
> Reply-To: Struts Developers List <st...@jakarta.apache.org>
> To: jakarta-struts-cvs@apache.org
> Subject: cvs commit:
>     jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/taglib/html
>      ELRadioTag.java
>
> dmkarr      2002/10/02 22:00:47
>
>   Modified:    contrib/struts-el/src/share/org/apache/strutsel/taglib/html
>                         ELRadioTag.java
>   Log:
>   Uncommented "onmousemove", "onmouseout", and "onmouseover" attribute handlers,
>   as I found the base tag was properly handling these.  The "onmouseup" handler
>   is still commented out, as the bug in the base tag isn't fixed yet.
>

You're more than welcome to fix the bug in the base tag :-).

Craig


--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>