Bug 38019

Summary: Text line-heights wrong
Product: poppler Reporter: Joshua Richardson <joshuarbox-junk1>
Component: generalAssignee: poppler-bugs <poppler-bugs>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium    
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments: top of page in acroreader
top of page in poppler
shows top of page after patched
bottom of page in acroreader
bottom of page in poppler
shows bottom of page after patched
Patch to fix vertical line-spacing issues
Patch to remove some debugging messages that I introduced with previous patch

Description Joshua Richardson 2011-06-06 22:09:17 UTC
Created attachment 47635 [details]
top of page in acroreader

Some lines overwrite each other.  Some paragraphs have too much spacing.
Comment 1 Joshua Richardson 2011-06-06 22:10:16 UTC
Created attachment 47636 [details]
top of page in poppler

attachment shows text overwriting itself
Comment 2 Joshua Richardson 2011-06-06 22:11:11 UTC
Created attachment 47637 [details]
shows top of page after patched

fixed!
Comment 3 Joshua Richardson 2011-06-06 22:12:24 UTC
Created attachment 47638 [details]
bottom of page in acroreader
Comment 4 Joshua Richardson 2011-06-06 22:14:31 UTC
Created attachment 47639 [details]
bottom of page in poppler

shows too much space between paragraphs (extraneous line breaks)
Comment 5 Joshua Richardson 2011-06-06 22:15:12 UTC
Created attachment 47640 [details]
shows bottom of page after patched

fixed!
Comment 6 Joshua Richardson 2011-06-07 09:57:14 UTC
Created attachment 47667 [details] [review]
Patch to fix vertical line-spacing issues
Comment 7 Joshua Richardson 2011-06-07 09:58:29 UTC
Created attachment 47668 [details] [review]
Patch to remove some debugging messages that I introduced with previous patch
Comment 8 Albert Astals Cid 2011-06-07 10:51:08 UTC
Hi Joshua, thanks for the patches, but please do not mark the bug as Fixed because it is not, it will be once we commit the patches to the git repository
Comment 9 Albert Astals Cid 2011-06-18 05:42:29 UTC
I've commited a cleaned version of your patches and it will be part of next poppler >= 0.17.x releases

http://cgit.freedesktop.org/poppler/poppler/commit/?id=d4af1c4ef46abf1f11b7215c7b144ce7bb7912eb

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.