W3C home > Mailing lists > Public > public-forms@w3.org > August 2007

Re: XForms 1.1 spec editor's drafts now available on public page (PR#91)

From: Ulrich Nicolas Lissé <unl@dreamlab.net>
Date: Wed, 01 Aug 2007 18:29:31 +0200
Message-ID: <46B0B4EB.6040906@dreamlab.net>
To: John Boyer <xforms-issues@mn.aptest.com>
CC: public-forms@w3.org

John,

I agree completely with the given resolutions.

Thanks,
Uli.

John Boyer wrote:
> Hi Uli,
> 
> Thanks for spotting these issues.  Resolutions are given in the inline comments.
>  Please see the editor's draft of XForms 1.1 available from the Forms WG home
> page and let us know if you have any further concerns about these issues.
> 
> Thanks,
> John Boyer
> 
>> Hello John,
>>
>> great work. Alas, some typos and other minor issues popped up during
>> fine reading. Maybe some of these are contained in XForms 1.0 editions
>> too, but I didn't check this.
>>
>> "3.3 The XForms Core Module"
>>
>> The table contains a wrong attribute definition for the submission
>> element. It says: "... method
>> ("post"|"get"|"put"|delete|"form-data-post"|"urlencoded-post"|QNameButNotNCNAME)
>>
>> ...."
> 
> Fixed.
> 
>> The word delete should have double quotes, since it is a fixed value
>> instead of a type.
>>
>> "3.3.1 The model Element"
>>
>> The sentence "This element is not an XForms Action, and has no
>> predefined behavior event-based behavior." has too many behaviors ;-)
>> Remove "behavior" after "predefined".
>>
> 
> The whole phrase containing the error is removed as it is ambiguous even once
> the error is corrected.
> 
>> "3.3.2 The instance Element"
>>
>> In the description of the resource attribute there is the sentence "If
>> the host language may treat failure to traverse the link as an exception
>> (4.5.2 The xforms-link-exception Event).", which is not really
>> understandable. Remove "If".
> 
> Fixed via changes to allow the src attribute into XForms directly (as opposed to
> being added by the host language).
> 
>> "4.5.5 The xforms-version-exception Event"
>>
>> For this event's context information a property "errorinformation" is
>> defined. Shouldn't this be named "error-information" instead to be
>> consistent with the naming scheme of other event's properties (like
>> "resource-uri" or "error-message")?
> 
> Fixed.
> 
>> "4.6.1 For input, secret, textarea, range, or upload Controls"
>>
>> The first bullet says "... has the "incremental="true" setting ...", the
>> second "... does not have the "incremental=true" setting". The double
>> quotes for the incremental attribute have to be fixed, and it should be
>> set in <code>-markup.
>>
> 
> Fixed.
> 
>> "4.6.3 For select or select1 Controls"
>>
>> Same as above.
> 
> Fixed.
> 
>> "7.6 The XForms Function Library"
>>
>> Replace "xforms-binding exception" with "xforms-binding-exception" in
>> the last sentence.
> 
> Fixed.
> 
>> "7.8.5 The index() Function"
>>
>> The second sentence "if the specified argument ..." should start uppercase.
>>
> 
> Fixed.
> 
>> "7.9.4 The digest() Function"
>>
>> All three expamples for this extension function still show
>> "hash-encode". Replace with "digest".
>>
> 
> Good catch.  Fixed.
> 
>> "8.1.5 The output Element"
>>
>> There are two Description paragraphs provided here. They are similar but
>> not identical. The second one seems more accurate, so please delete the
>> first one.
>>
> 
> Fixed.
> 
>> "8.3.1 The filename Element"
>>
>> In the description of a then following example
>> "mail/attachment@filename" and "mail/attachment@mediatype" refer to
>> attribute nodes. The correct XPath syntax "mail/attachment/@filename"
>> and "mail/attachment/@mediatype" should be used.
>>
> 
> Good catch. Fixed.
> 
>> "9.3.1 The repeat Element"
>>
>> In the paragraph after the special attributes description the second
>> sentence "if an element of the collection ..." should start uppercase.
>>
> 
> Fixed.
> 
>> "9.3.5 The insert Action"
>>
>> The 5th example ("Inserting into a repeat, whether or not it is empty")
>> refers to "repeat R". In the code snippet there is a call to index()
>> function in <xf:insert ... at="index('R')"/> as well as a <xf:setfocus
>> control="R"/>. However, the repeat is missing id="R" which might confuse
>> readers. Maybe it could even be renamed to something more
>> self-explanatory, e.g. "PurchaseOrderRepeat".
>>
> 
> Added id="R"
> 
>> "9.3.7 The setindex Element"
>>
>> In the paragraph after the special attributes description the third
>> sentence "if the index evaluates to NaN ..." should start uppercase.
>>
> 
> Fixed.
> 
>> "9.3.8 Repeat Processing"
>>
>> In list item 2. the second sentence "if the initial startindex ..."
>> should start uppercase.
>>
> 
> Fixed.
> 
>> "10.10.2 Processing of the load Action"
>>
>> In the second paragraph "... link between th load element and the remote
>> resource" replace "th " with "the ".
> 
> Fixed.
> 
>> "10.15 Iteration of XForms Actions"
>>
>> I don't speak English natively, but the sentence "Counter and Accumlator
>> Variables are Created in Instance Data to Sum a Selection of Values
>> Chosen by the User" hurts my eyes. Shouldn't this simply be "Counter and
>> accumlator variables are created in instance data to sum a selection of
>> values chosen by the user." ??
>>
> 
> You're right.  It's not a title, so it shouldn't be in title case.
> Fixed.
>  
>> "11.2 The xforms-submit Event"
>>
>> In "The response returned from the submission is applied as follows:"
>> list, the second bullet's second sentence "if the parse fails ..."
>> should start uppercase.
>>
> 
> Fixed.
> 
>> "11.19.3 SOAP HTTP Binding"
>>
>> In the second list the first bullet "the Content-type HTTP header is
>> change to text/xml" should be changed to contain "changed".
> 
> Fixed.
> 
>> Regards,
>> Uli.
>> -- 
>> Ulrich Nicolas Liss銾 
>>
>>


-- 
Ulrich Nicolas Lissé
Received on Wednesday, 1 August 2007 16:29:34 UTC

This archive was generated by hypermail 2.3.1 : Tuesday, 1 October 2013 22:06:44 UTC