Hi, On Monday 18 February 2013, meik michalke wrote: > > > - rk.XML.switch()
[...]
> i've simplified it now, but it's a little different from what you suggested:
[...]
> i preferred it this way because it's a little more consistent with the
> output, especially since "standard" is supposed only to be valid for
> <case> nodes (therefore i'd find "standard=TRUE/FALSE" to indicate
> <true/false> nodes a bit confusing). it's a bit more to write for <case>
> stuff, but at the same time, the original intention of a plugin author is
> documented.
>
> what do you think?
Looks good to me. Note that the original reason to have four different tags -
true, false, case, default - as child elements of the switch is simply that
there is no good way in an XML argument, to differentiate the string "TRUE"
from a logical TRUE, or an empty string from "nothing". But your points about
readability and consistency are quite valid.
> the <optionset> docs do also still need more examples.
Ok, optionset next. I tried translating the first optionset example from the
"primary" documentation into rkwarddev. This is what I got:
firstname <- rk.XML.input ("Given name(s)")
lastname <- rk.XML.input ("Family name")
genderselect <- rk.XML.radio ("Gender", options = list ("Male"=c(val="m"),
Female=c(val="f")))
rk.XML.optionset (
content = list (rk.XML.optiondisplay (), rk.XML.row (firstname,
lastname,
genderselect)),
optioncolumn = list (
rk.XML.optioncolumn("firstnames", connect=firstname,
modifier="text"),
rk.XML.optioncolumn("lastnames", connect=lastname,
modifier="text"),
rk.XML.optioncolumn("genders", connect=genderselect)
)
)
Perhaps you want to add this as a code example. The generated code was correct
for this. I did not test extensively, beyond this. However, I did try a few of
parameters, and the ones I tested worked correctly, too.
Regards
Thomas
signature.asc
Description: This is a digitally signed message part.
------------------------------------------------------------------------------ Everyone hates slow websites. So do we. Make your web apps faster with AppDynamics Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_feb
_______________________________________________ RKWard-devel mailing list RKWard-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/rkward-devel
