<br><div class="gmail_quote">On Wed, May 6, 2009 at 12:56 PM, Ed Burns <span dir="ltr"><<a href="mailto:Ed.Burns@sun.com">Ed.Burns@sun.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
>>>>> On Tue, 05 May 2009 20:40:50 -0400, Kito Mann <<a href="mailto:kito.mann@VIRTUA.COM">kito.mann@VIRTUA.COM</a>> said:<br>
<br>
KM> Hello Ed,<br>
KM> I think you and Roger have done a wonderful job putting this all together,<br>
KM> especially with all of the different artifacts. A few comments:<br>
<br>
KM> PLD Docs<br>
<br>
KM> * Since we've decided to call it the VDL, why are these still called<br>
KM> PDLDocs?<br>
<br>
That's not worth fixing right now.<br>
<br>
KM> * The <composite:attribute> "method-signature" should probably be renamed<br>
KM> "methodSignature" to be more consistent with the other attributes<br>
<br>
We'll have to live with that, unfortunately.<br>
<br>
KM> PDF<br>
<br>
KM> Whole document -- "page description language" is still used in a few places<br>
KM> -- a Search and Replace needs to be performed, and this will create some<br>
KM> minor changes for the RI (for example, the configuration in 11.4.5)<br>
<br>
Thanks, I thought I got those, but I guess not. I hope some changes<br>
were not over-ridden due to the inability of version control to deal<br>
with .fm files correctly.<br>
<br>
Fixed.<br>
<br>
KM> * Section 3.6.1.6, formatting is messed up<br>
<br>
Fixed<br>
<br>
KM> * Section 5.6.1, p 5-15: "resource" is spelled incorrectly when describing<br>
KM> getValue()<br>
<br>
Fixed.<br>
<br>
KM> * Section 5.6.2 doesn't mention "flash" as an implicit variable, and I don't<br>
KM> see any other reference to how it is resolved in the PDF. The JavaDocs for<br>
KM> Flash say its an implicit variable called "flash".<br>
<br>
Fixed.<br>
<br>
KM> In addition, I would<br>
KM> expect "flash" to be "flashScope" like the other scope (even though it has<br>
KM> special semantics).<br>
<br>
For ease of use and convention with Rails.</blockquote><div><br>In that case, I think we should _also_ support flashScope. Otherwise it looks like we made a mistake to non-Rails developers. <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<br>
KM> And why is flash a property of the ExternalContext instead of the<br>
KM> FacesContext?<br>
<br>
Portlet requirements.<br>
<br>
KM> * Section 13.1.1.1 -- As I've mentioned several times, it's silly for us not<br>
KM> to have a simple @AjaxResource annotation that encapsulates the standard<br>
KM> librarry name and its location. Every component developer is going to have<br>
KM> to do this themselves.<br>
<br>
That's a new feature for 2.1<br>
<br>
KM> * Section 13.1.1.3 -- Same issue as 13.1.1.1 -- should be a<br>
KM> <h:outputAjaxScript/> element<br>
<br>
That's a new feature for 2.1</blockquote><div><br>I'll add these the bug tracker, but I don't see why they can't be in a maintenance release, since it's really just syntactic sugar. <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<br>
KM> * Section 13.1.1.3 -- Should say "The View Declaration Language Approach"<br>
<br>
Fixed.<br>
<br>
Thanks.<br>
<br>
Another draft, the final, forthcoming.<br>
<font color="#888888"><br>
Ed<br>
</font></blockquote></div><br>