Menu

#74 Remove unnecessary table from "html/html_mail_*.php"

closed-fixed
None
5
2006-10-02
2006-09-27
Tim Gerundt
No

This patch remove the unnecessary table from
"html_mail_*.php" for the mail_body. Now a table is
only used for the attachments in the "View header"
view. The layout should be the same!
I also add two "\n" for a nicer html source code and
fix a XHTML error (& -> &) in "html/html_mail_top.php".

By the way: Would it not be easier, if you merge
"html_mail_top.php", "html_mail_header.php" and
"html/html_mail_bottom.php" to one file?

Greetings,
Tim Gerundt

Discussion

  • Tim Gerundt

    Tim Gerundt - 2006-09-27

    original and altered files

     
  • Tim Gerundt

    Tim Gerundt - 2006-09-27

    patch file

     
  • Tim Gerundt

    Tim Gerundt - 2006-09-27

    Logged In: YES
    user_id=652377

    I forgot: The files should have now Unix line feeds! I found
    the option in WinCVS! ;-)

     
  • Anonymous

    Anonymous - 2006-10-02
    • assigned_to: nobody --> goddess_skuld
    • status: open --> closed-fixed
     
  • Anonymous

    Anonymous - 2006-10-02

    Logged In: YES
    user_id=529507

    Fixed in CVS.

    It'll be included in next release.

    But you can download daily snapshots at :
    http://nocc.sourceforge.net/download/

    Thanks for the patch.

     

Log in to post a comment.

MongoDB Logo MongoDB