Bug 738689 - Screen produces different vertical spaces when surrounded by paras
Summary: Screen produces different vertical spaces when surrounded by paras
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Publican
Classification: Community
Component: publican
Version: 2.7
Hardware: Unspecified
OS: Unspecified
low
unspecified
Target Milestone: 3.0
Assignee: Jeff Fearn 🐞
QA Contact: Ruediger Landmann
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-09-15 14:31 UTC by Martin Prpič
Modified: 2012-10-31 03:11 UTC (History)
4 users (show)

Fixed In Version: 3.0.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-10-31 03:11:50 UTC


Attachments (Terms of Use)
Proposed patch (596 bytes, patch)
2011-10-06 15:40 UTC, Jaromir Hradilek
no flags Details | Diff

Description Martin Prpič 2011-09-15 14:31:52 UTC
Description of problem:

When screen is surrounded by paras, the horizontal spacing after screen is smaller than the one before screen.

Version-Release number of selected component (if applicable):
~]$ rpm -q publican
publican-2.7-1.fc15.noarch

How reproducible:
Always.

Steps to Reproduce:
1. Build a document with the following:

<para>
Blah blah blah blue blue blue red red red
</para>
<screen>
command 1; command 2; command 3;
command 3; command 5; command 6;
</screen>
<para>
Blah blah blah blue blue blue red red red
</para>

2. Open and view the screen portion.
  
Actual results:

The horizontal space between the first para and screen is larger than the horizontal space between screen and the second para.

Expected results:

The horizontal spaces should be of equal size. Right now, this can be achieved by including an empty para after screen:

<para>
Blah blah blah blue blue blue red red red
</para>
<screen>
command 1; command 2; command 3;
command 3; command 5; command 6;
</screen>
<para>
</para>
<para>
Blah blah blah blue blue blue red red red
</para>

However, that's an ugly hack and should not be used :)

Additional info:
As far as I know, this also applies to programlisting.

Comment 1 Martin Prpič 2011-09-15 15:15:16 UTC
Blargh, of course I meant vertical!

Comment 2 Jaromir Hradilek 2011-10-06 15:40:26 UTC
Created attachment 526733 [details]
Proposed patch

Comment 3 Jeff Fearn 🐞 2011-10-10 10:49:28 UTC
Applied patch to branches/publican-2x and trunk.

Committed revision 1907.

Comment 4 Misha H. Ali 2012-04-22 21:23:21 UTC
Verified vertical space between first and second (screen) para is equal to vertical space between second (screen) and third para.

Verified on:

OS: Fedora 16
Publican: 3.0
Release designation: publican-3.0-0.fc16.t166.src.rp


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