Bug 531686 - Callout list rendering issue
Summary: Callout list rendering issue
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Publican
Classification: Community
Component: publican
Version: 1.6
Hardware: All
OS: Linux
high
high
Target Milestone: ---
Assignee: Jeff Fearn 🐞
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-10-29 05:17 UTC by BenC
Modified: 2010-11-24 04:16 UTC (History)
4 users (show)

Fixed In Version: 1.3-0.fc12
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-12-10 04:14:28 UTC
Embargoed:


Attachments (Terms of Use)
html example of issue (rendering problem) (36.77 KB, image/png)
2009-10-29 05:17 UTC, BenC
no flags Details
HTML example of fixed output (19.12 KB, image/png)
2009-11-25 22:40 UTC, Jeff Fearn 🐞
no flags Details
PDF example of fixed output (43.91 KB, image/png)
2009-11-25 22:41 UTC, Jeff Fearn 🐞
no flags Details

Description BenC 2009-10-29 05:17:39 UTC
Created attachment 366553 [details]
html example of issue (rendering problem)

Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


h<programlistingco>
<areaspec>
<areaset id="ex.plco.const" coords="">
  <area id="ex.plco.c1" coords='4'/>
  <area id="ex.plco.c2" coords='8'/>
</areaset>
<area id="ex.plco.ret" coords='12'/>
<area id="ex.plco.dest" coords='12'/>
</areaspec>
<programlisting>
sub do_nothing_useful {
    my($a, $b, $c);

    $a = new A;
    
    $a->does_nothing_either();

    $b = new B; 

    $c = "frog";

    return ($a, $c);
}
</programlisting>
<calloutlist>
<callout arearefs="ex.plco.const">
<para>
These are calls to the constructor <function>new</function> in the object
classes.
</para>
</callout>
<callout arearefs="ex.plco.ret">
<para>
This function returns a two-element list.
</para>
</callout>
<callout arearefs="ex.plco.dest">
<para>
The <emphasis>destructor</emphasis> (<function>DESTROY</function>) for
the object <literal>$b</literal> will be called automatically for this
object since there can be no other references to it outside this function.
</para>
</callout>
</calloutlist>
</programlistingco>


Additional info:

The resultant html and pdf are unsightly and inconsistent. Large numbers in html and small numbers in the PDF.

Comment 1 Jeff Fearn 🐞 2009-11-24 23:18:50 UTC
Clearly Ben is running an old publican because on publican 1.x you get:

No insertCallouts function is available.
 at /usr/lib/perl5/vendor_perl/5.8.8/Publican/Builder.pm line 738

Oopsy!

Need to implement this callout function similar to how the highlight function is done in Builder.pm.

Then make them pretty.

P.S. Rudi, is 1.x still going to happen for F11?

Comment 2 Jeff Fearn 🐞 2009-11-25 22:40:27 UTC
Created attachment 373851 [details]
HTML example of fixed output

Comment 3 Jeff Fearn 🐞 2009-11-25 22:41:01 UTC
Created attachment 373852 [details]
PDF example of fixed output

Comment 4 Jeff Fearn 🐞 2009-11-25 22:44:24 UTC
Added insertCallouts xstl function. Fixed HTML & PDF layouts.

Comment 5 Fedora Update System 2009-12-08 02:14:24 UTC
publican-1.3-0.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/publican-1.3-0.fc12

Comment 6 Fedora Update System 2009-12-08 02:17:48 UTC
publican-1.3-0.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/publican-1.3-0.fc11

Comment 7 Fedora Update System 2009-12-10 04:13:24 UTC
publican-1.3-0.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 8 Fedora Update System 2009-12-10 04:26:16 UTC
publican-1.3-0.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.