On Fri, 31 Dec 2004, Matthias Wessendorf wrote:

<snip/>

Bill Dudney added <x:div/>
http://incubator.apache.org/myfaces/tlddoc/x/div.html

A couple of comments on this:

1) The docs for this tag say that either style or styleClass must be specified. Why is this necessary? If the div is simply there for use in manipulation by JavaScript, there would be no style involved.

2) The attribute documentation needs another column (or some other means) to specify the default value for attributes. For example, the docs for <x:div> give no indication of what happens if I don't specify values for the 'rendering' and 'transient' attributes. While this could be added in the Description, as is done for 'id', it's all too easy to forget to add it, which is probably what happened for the others in this tag.

--
Martin Cooper



Reply via email to