OASIS Mailing List ArchivesView the OASIS mailing list archive below
or browse/search using MarkMail.

 


Help: OASIS Mailing Lists Help | MarkMail Help

office message

[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]


Subject: Errata in ODF spec 1.1


Hello all,

Three corrections in Section 4.6.4 Deletion, pages 80 and 81.

Correction #1 - Page 80
-------------
<quote>
To reconstruct the text before the deletion took place, do:

* If the change mark is inside a paragraph, insert the text content of
the <text:deletion> element as if the beginning <text:p> and final
</text:p> tags were missing.
* If the change mark is inside a header, proceed as above, except adapt
the inserted tags to math their new counterparts.
* Otherwise, simply copy the text content of the <text:deletion> element
in place of the change mark.
</quote>

Second bullet point: math -> match

Correcton #2: - page 81
-------------
The third example says:
//------ snip
      <text:p>abcdef</text:p>
      <text:change text:id="example/>
      <text:p>ghijkl</text:p>
This becomes:
      <text:p>abcdef</text:p>
      <text:h>Hello</text:h>
      <text:h>World!</text:h>
      <text:p>ghijkl</text:p>
//------

The tags around Hello and World should be text:p not text:h. So the last
4 lines should read:

      <text:p>abcdef</text:p>
      <text:p>Hello</text:p>
      <text:p>World!</text:p>
      <text:p>ghijkl</text:p>


Correcton #3: - page 81
-------------
Very last example of this section. Example says:

//------ snip start ----//
The change that took place from
      <text:p>abc</text:p>
      <text:h>Hello</text:h>
      <text:h>World!</text:h>
      <text:p>def</text:p>
to
      <text:p>abc<text:change text:id="example/>def</text:p>

would be represented as:

      <text:changed-region text:id="example">
           <text:deletion>
               <office:change-info>...</office:change-info>
               <text:p/>
               <text:p>Hello</text:p>
               <text:p>World!</text:p>
               <text:p/>
           </text:deletion>
      </text:changed-region>
//------ snip end ------//

The <text:h> tags on the first block don't match the <text:p> tags in
the changeset. There are two ways to fix this.

1) Change the first block to:
      <text:p>abc</text:p>
      <text:p>Hello</text:p>
      <text:p>World!</text:p>
      <text:p>def</text:p>

2) Change the changeset to:
           <text:deletion>
               <office:change-info>...</office:change-info>
               <text:p/>
               <text:h>Hello</text:h>
               <text:h>World!</text:h>
               <text:p/>
           </text:deletion>


Either of these is a valid solution, but we may need to change the
explanatory text around here to make the examples fit well.

My suggestion is:

* Change the preceding text from this:

  "If, in the first two cases, the deletion contains complete
  paragraphs, then additional empty paragraphs must be put into
  the <text:deletion> element to achieve the desired result."

To this:

  "In some cases additional empty paragraphs must be put into
  the <text:deletion> element to achieve the desired result.
  For example:"

* And then update the changeset to use <text:h>.

It seems to me that this is what the original writer had in mind. The
example seems designed to show how to create a changeset that contains
<text:h> when the change mark is inside a <text:p>.

Cheers,
Daniel.
-- 
http://opendocumentfellowship.org
  "The reasonable man adapts himself to the world; the
  unreasonable man tries to adapt the world to himself.
  Therefore all progress depends on unreasonable men."
        -- George Bernard Shaw

This is a digitally signed message part



[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]