Hi everybody,

I've been meaning to bring this up for a while now. The new-for-5.3
Checklist component has -- for me -- a couple of minor annoyances:

1) The label for each checkbox is positioned *before* (that is, to the
left of) the checkbox. I think it is much more common to want the
label *after* (to the right of) the checkbox.

    See 
http://tapestry.apache.org/current/apidocs/org/apache/tapestry5/corelib/components/checklist_ref.png

2) The <label> for each checkbox is not connected to the checkbox with
a "for" attribute, so you can't click on the label to check or uncheck
the box.

3) This component takes an "encoder" parameter but does not support a
ValueEncoder configured to be provided automatically by Tapestry.

Do others agree that these should be changed?

Once the 5.3 final release is out, I guess #1 can't be changed without
potentially making somebody unhappy, so that's why there might be some
urgency to do at least #1 before 5.3 is released. For someone with a
working Tapestry build environment (mine is broken currently) #1 and
#2 are of course very easy to fix.

http://tapestry.apache.org/current/apidocs/org/apache/tapestry5/corelib/components/Checklist.html

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to