Bug 738689

Summary: Screen produces different vertical spaces when surrounded by paras
Product: [Community] Publican Reporter: Martin Prpič <mprpic>
Component: publicanAssignee: Jeff Fearn 🐞 <jfearn>
Status: CLOSED CURRENTRELEASE QA Contact: Ruediger Landmann <rlandman+disabled>
Severity: unspecified Docs Contact:
Priority: low    
Version: 2.7CC: jfearn, jhradile, mhusnain, rlandman+disabled
Target Milestone: 3.0   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 3.0.0 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-10-31 03:11:50 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Proposed patch none

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