Hi !

I have just finished reviewing the new docs from Andre. All in all everything
went fine, only a few minor changes were needed to get it all working correctly.
I also merged in all the changes from the last days, which were missing in Andres
docs.


I think we are on a very good way!

Below some *minor* issues I spotted while reviewing the docs. I don't think they
should keep us from committing.


a) style/lang/ ja.xml sv.xml
We now have some more language-dependent phrases which should be implemented
by the authors.

b) style/lang/ pt-br.xml
Is coming the next days, as far as I know Luiz Rocha is working on this.

c) We now have some more images in manual/images:
down.gif
feather.gif
left.gif
right.gif
up.gif

These should be converted to PNG too (as all the others).

d) I had to make some minor changes to common.dtd to get the new docs
validating correctly: for example I defined two new elements to support the
newly used <cite>-, <dfn>-tags.

e) The example-boxes use <div class="example"><pre>...</pre></div> to define
the bg-color. Unfortunately Mozilla and IE (didn't check the others) don't
wrap the text within the <pre>-tags, since it is pre-defined, ha. With
smaller window-sizes we therefore get lines which reach over the background box.


To see this live, try http://test.perlig.de/apdoc/manual/mod/mod_rewrite.html
with a window-size smaller than 800x600 (scroll down to find a problematic
place).



Joshua, I also had to change build.xml, can you please commit the attached patch; thanks ;-) I will go on and commit the whole stuff in the next hour if nobody raises any objections.

Erik
Index: build.xml
===================================================================
RCS file: /home/cvspublic/site-tools/httpd-docs-build/build.xml,v
retrieving revision 1.7
diff -u -r1.7 build.xml
--- build.xml   4 Sep 2002 15:47:34 -0000       1.7
+++ build.xml   5 Sep 2002 22:10:27 -0000
@@ -10,7 +10,7 @@
     <patternset id="excludes">
         <exclude name="build/*.xml" />
         <exclude name="mod/allmodules.xml*" />
-        <exclude name="style/*.xml" />
+        <exclude name="style/lang/*.xml" />
     </patternset>
  
     <patternset id="en-allxml">
@@ -146,4 +146,3 @@
         </xmlvalidate>
     </target>
 </project>
-


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to