Patch for HTML export (bug#461?)


Subject: Patch for HTML export (bug#461?)
d.rose@salford.ac.uk
Date: Fri Dec 03 1999 - 06:36:10 CST


Oops! Sent this to the wrong address first time around.

2nd time lucky...

        -D

ps. hdiff.out is diff file for .h file
    cppdiff.out is diff file for .cpp file

attached mail follows:


I've made the following changes to files ie_exp_HTML.h and ie_exp_HTML.cpp
(both in src/wp/impexp/xp directory) so that HTML export is a little better.

This is the first patch I've submitted so there might be a couple of gotcha's
to watch out for (comments would be appreciated so I can improve my
patch-submitting mechanisms :>)

1. Diff produced with -u switch.

2. Before changes made, original files were moved to <filename>.orig -
   diffs taken with 'diff -u <filename>.orig <filename>'. Not sure if
   this affects the way a patch can be applied or not (how is this
   normally done?)

3. I think I've stuck to coding standards, you may like to check...

What's changed with the code in general that may or may not have an impact:

1. "Normal" html paras now start with <p> and end with </p> (used to start
   with <P> end with </p> which my XML friends tell me is not cool...)

2. Text styles Heading 1, Heading 2, Heading 3 exported as HTML h1, h2, h3

3. Text style Plain Text => HTML <pre/>

4. Text style Block Text => HTML <blockquote/>

Something I'd like to ask is: is it possible _ever_ to nest these style
types? I know headings don't nest, but how about Block/Plain text? They
don't _appear_ to nest. The code will fail (i.e. stick in the wrong
end-tags) if nesting does occur.

That's it for now...

        -Duncan





This archive was generated by hypermail 2b25 : Fri Dec 03 1999 - 06:33:03 CST