Hi Johan,

neither really knowing the new implementation yet nor having much
experience with wicket in general (just learning), please allow me to
suggest a tiny improvement: I think, it's better to keep the order of
the constructor parameters and add optional parameter at the end.

So it should IMHO be

 DropDownChoice(String id, IModel model, List source)
 DropDownChoice(String id, IModel model, List source, IChoiceRenderer renderer)

Another point: When I experimented with the old implementation of
Choice, I tried to setup something that would display a list of
radiobuttons but should display multiple columns, i.e.

 ( ) bla       bla bla
 ( ) blabla    blaha bla
 ( ) bla2      blabla blah
 
this wasn't really possible because I couldn't emit the required <td>
and </td> tags and I had to use a difficult workaround using a
ListView. It would be nice if you could somehow switch off
markup-escaping.

Another problem was to print a euro sign (€) which shall be added to
the HTML as &euro; but which is always escaped as &amp;euro;. And
directly embedding the euro sign didn't work either - probably because
of encoding problems which get hopefully sorted out for 1.1.

And a question: How does a MultipleValueChoice (for example a list of
check boxes) work with your new API.  Is it possible to pass a List as
Model?

Regards,

-- 
Stefan Matthias Aust // Truth until paradox
N¬HS^µéšŠX¬²š'²ŠÞu¼­…àè2ŠÞZ+aæ¥Áæâ�ªájš^ž)à&ér׆­ñ©�
?õÕ©„
5žŠ{âµì¨ºÚÆ™h­ëazV­zËb�Ûš•Ê+y©Ýv楂¶©†',µç!ž‰hƒ&­¶¬~·ž¢w¡¢êÞ½é톋-yÖòð
©Ý5R 
è­è"²×«¾+"¶m§ÿðÃir‰¿‚�ݹ©py¸§jµ¢rG­ºÇ«™¨¥Šx%ŠËV‰É¶ë®X¬¶Ë(º·~Šàzw­†Ûi³ÿåŠËl²‹«qçè®§zßåŠËlþX¬¶)ߣü"rG­ºÇ«

Reply via email to