[ 
https://issues.apache.org/jira/browse/WICKET-1271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12562518#action_12562518
 ] 

verhage edited comment on WICKET-1271 at 1/25/08 7:49 AM:
---------------------------------------------------------------

More elegant way of fetching descendant input elements, without cumbersome loop

/edit:
Grmbl, nevermind my patch, doesn't seem to work in firefox...

/edit 2:
I stand corrected, my patch seems to also work within firefox. However, if I 
declare my HTML like:

<table>
  <tr>
    <span wicket:id="radiogroup" id="someid">
      <td><input wicket:id="radio1" /></td>
      <td><input wicket:id="radio1" /></td>
    </span>
  </tr>
</table>

won't work as firefox does not accept the span tag between <tr> and <td>, and 
therefore will return a single node of #text when calling 
document.getElementById('someid').childNodes. Placing the span outside the 
table works. IE doesn't seem too picky about this.

      was (Author: verhage):
    More elegant way of fetching descendant input elements, without cumbersome 
loop

/edit:
Grmbl, nevermind my patch, doesn't seem to work in firefox...
  
> Script in RenderHead method of AjaxFormChoiceComponentUpdatingBehavior needs 
> fixing
> -----------------------------------------------------------------------------------
>
>                 Key: WICKET-1271
>                 URL: https://issues.apache.org/jira/browse/WICKET-1271
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.3.0-final
>            Reporter: Dipu C Seminlal
>            Assignee: Igor Vaynberg
>             Fix For: 1.3.1
>
>         Attachments: 
> AjaxFormChoiceComponentUpdatingBehavior_patch_revision_610340.txt, patch.txt
>
>
> The script in the renderHead method of 
> AjaxFormChoiceComponentUpdatingBehavior assumes that the  Radio Component 
> will be the immediate child of the RadioGroup Component, But in some or in 
> most of the cases that is not correct.
> That is not the case even in the Wicket Examples, in examples the hierarchy 
> is RadioGroup/ListView/Radio.
> I have maodied the rendered script to work for RadioGroup/ListView/Radio 
> hierarchy. 
> And the Patch is attached.
> Regards
> Dipu

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to