Bug 28276

Summary: The text layout information is necessary for evince accessibility support
Product: poppler Reporter: Daniel Garcia <dani>
Component: glib frontendAssignee: 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: patch to add a function to glib for get the text position in a page
patch to add a function to glib for get the text position in a page
patch to add a function to glib to get text position in a page
patch to add a function to glib to get text position in a page
patch to add a function to glib to get text position in a page.

Description Daniel Garcia 2010-05-27 01:43:47 UTC
Created attachment 35877 [details] [review]
patch to add a function to glib for get the text position in a page

I'm implementing a11y support for evince and I need the text position information. In glib poppler frontend there is no way to get the text layout from a page.

It's related with evince bug:
https://bugzilla.gnome.org/show_bug.cgi?id=309015
Comment 1 Daniel Garcia 2010-06-01 06:35:26 UTC
Created attachment 35985 [details] [review]
patch to add a function to glib for get the text position in a page

I attached a better patch, the mapping now works well with multicolumn pdf files.
Comment 2 Daniel Garcia 2010-06-07 04:51:37 UTC
Created attachment 36098 [details] [review]
patch to add a function to glib to get text position in a page

Using prepend instead of append commit added. I changed the glist append and used prepend and reverse for eficience.
Comment 3 Albert Astals Cid 2010-06-07 05:29:01 UTC
Not fixed until patch is commited.
Comment 4 Daniel Garcia 2010-06-15 07:15:53 UTC
Created attachment 36288 [details] [review]
patch to add a function to glib to get text position in a page

I changed the patch to return an array of PopplerRectangle instead of PopplerMapping. Now there's no duplicated information and spaces are well posicioned. And the wordlist is freed also.
Comment 5 Daniel Garcia 2010-06-15 08:01:40 UTC
Created attachment 36289 [details] [review]
patch to add a function to glib to get text position in a page.

I removed trailing white spaces fix without patch relation.
Comment 6 Carlos Garcia Campos 2010-06-16 02:57:36 UTC
Pushed the patch with some minor changes. Thank you very much.

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.