Bug 28276 - The text layout information is necessary for evince accessibility support
Summary: The text layout information is necessary for evince accessibility support
Status: RESOLVED FIXED
Alias: None
Product: poppler
Classification: Unclassified
Component: glib frontend (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: poppler-bugs
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-27 01:43 UTC by Daniel Garcia
Modified: 2010-06-16 02:57 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
patch to add a function to glib for get the text position in a page (5.47 KB, patch)
2010-05-27 01:43 UTC, Daniel Garcia
Details | Splinter Review
patch to add a function to glib for get the text position in a page (5.47 KB, patch)
2010-06-01 06:35 UTC, Daniel Garcia
Details | Splinter Review
patch to add a function to glib to get text position in a page (5.49 KB, patch)
2010-06-07 04:51 UTC, Daniel Garcia
Details | Splinter Review
patch to add a function to glib to get text position in a page (4.03 KB, patch)
2010-06-15 07:15 UTC, Daniel Garcia
Details | Splinter Review
patch to add a function to glib to get text position in a page. (3.77 KB, patch)
2010-06-15 08:01 UTC, Daniel Garcia
Details | Splinter Review

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.