>>>>> On Tue, 05 May 2009 20:40:50 -0400, Kito Mann <kito.mann@VIRTUA.COM> said:

KM> Hello Ed,
KM> I think you and Roger have done a wonderful job putting this all together,
KM> especially with all of the different artifacts. A few comments:

KM> PLD Docs

KM> * Since we've decided to call it the VDL, why are these still called
KM> PDLDocs?

That's not worth fixing right now.

KM> * The <composite:attribute> "method-signature" should probably be renamed
KM> "methodSignature" to be more consistent with the other attributes

We'll have to live with that, unfortunately.

KM> PDF

KM> Whole document -- "page description language" is still used in a few places
KM> -- a Search and Replace needs to be performed, and this will create some
KM> minor changes for the RI  (for example, the configuration in 11.4.5)

Thanks, I thought I got those, but I guess not.  I hope some changes
were not over-ridden due to the inability of version control to deal
with .fm files correctly.

Fixed.

KM> * Section 3.6.1.6, formatting is messed up

Fixed

KM> * Section 5.6.1, p 5-15: "resource" is spelled incorrectly when describing
KM> getValue()

Fixed.

KM> * Section 5.6.2 doesn't mention "flash" as an implicit variable, and I don't
KM> see any other reference to how it is resolved in the PDF. The JavaDocs for
KM> Flash say its an implicit variable called "flash".

Fixed.

KM> In addition, I would
KM> expect "flash" to be "flashScope" like the other scope (even though it has
KM> special semantics).

For ease of use and convention with Rails.

KM> And why is flash a property of the ExternalContext instead of the
KM> FacesContext?

Portlet requirements.

KM> * Section 13.1.1.1 -- As I've mentioned several times, it's silly for us not
KM> to have a simple @AjaxResource annotation that encapsulates the standard
KM> librarry name and its location. Every component developer is going to have
KM> to do this themselves.

That's a new feature for 2.1

KM> * Section 13.1.1.3 -- Same issue as 13.1.1.1 -- should be a
KM> <h:outputAjaxScript/> element

That's a new feature for 2.1

KM> * Section 13.1.1.3 -- Should say "The View Declaration Language Approach"

Fixed.

Thanks.

Another draft, the final, forthcoming.

Ed