[ docbook-Bugs-2844927 ] HTML: Validity error for callout bugs

[SourceForge.net <no...@so...>]

Bugs item #2844927, was opened at 2009-08-26 13:21
Message generated for change (Comment added) made by sf-robot
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=373747&aid=2844927&group_id=21935

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: XSL
Group: output: HTML
>Status: Closed
Resolution: Fixed
Priority: 5
Private: No
Submitted By: Simon Kennedy (sffjunkie)
Assigned to: Robert Stayton (bobstayton)
Summary: HTML: Validity error for callout bugs

Initial Comment:
When outputting callout bugs for programlisting, screen etc. the bugs are output as IMG elements within a PRE element. IMG elements are disallowed within PRE by the html DTD (even the 'loose' one).

----------------------------------------------------------------------

>Comment By: SourceForge Robot (sf-robot)
Date: 2010-09-29 19:05

Message:
This Tracker item was closed automatically by the system. It was
previously set to a Pending status, and the original submitter
did not respond within 14 days (the time period specified by
the administrator of this Tracker).

----------------------------------------------------------------------

Comment By: Robert Stayton (bobstayton)
Date: 2010-08-09 18:59

Message:
I changed verbatim output such that if $make.clean.html is nonzero, then
<pre> is replaced with a <div> with class attribute, and default styles are
added in docbook.css.  So if someone wants valid callout bugs, they need to
set $make.clean.html.

----------------------------------------------------------------------

Comment By: Robert Stayton (bobstayton)
Date: 2010-08-09 18:59

Message:
A fix for this issue has been added to the current codebase.
Please test the fix with the latest snapshot from:

  http://docbook.sourceforge.net/snapshots/


----------------------------------------------------------------------

Comment By: Robert Stayton (bobstayton)
Date: 2009-11-28 18:08

Message:
This solution sounds ok to me.  I'm wondering if the CSS style for the div
solution should be included in the output, though.  If we don't, then the
listing will be scrambled by default.  Putting it on the div itself with a
style attribute is one option, and another is putting  the style with
selector in the head element.

This item touches on the bigger subject of modernizing DocBook's HTML
output. I envision an option that generates clean HTML and a default CSS
stylesheet, unless  a param specifies the user's own stylesheet.  Then the
programlisting style could go there.

----------------------------------------------------------------------

Comment By: Mauritz Jeanson (mzjn)
Date: 2009-11-20 19:46

Message:
Feel free to comment. Is my suggested solution OK?

----------------------------------------------------------------------

Comment By: Mauritz Jeanson (mzjn)
Date: 2009-10-22 16:40

Message:
Using <div> instead of <pre> would make the callouts valid. A possible
solution is to use the make.valid.html parameter to decide whether <div> or
<pre> should be emitted (in verbatim.xsl). Something like this:

 <xsl:choose>
   <xsl:when test="$make.valid.html !=0 and $callout.graphics !=0 and 
	      (parent::programlistingco or parent::screenco or
child::co)">div</xsl:when>
   <xsl:otherwise>pre</xsl:otherwise>
 </xsl:choose>

When <div> is used, additional CSS is required to format the content
properly:

 div.programlisting {white-space: pre;
                     font-family: monospace; }

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=373747&aid=2844927&group_id=21935