[Poppler-bugs] [Bug 94517] page.text() does not take page orientation into account

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=94517

--- Comment #10 from Albert Astals Cid <aacid at kde.org> ---
(In reply to jalanpalmer from comment #9)
> (In reply to Albert Astals Cid from comment #8)
> > I'm a bit hesitant about that patch as since it would break the people that
> > are passing the coordinates to make it work like it works right now.
> 
> That's a good point. Maybe best to keep it how it is to avoid breaking any
> users.
> 
> > so maybe the only thing we need to do is actually do a switch of coords in
> > the 
> >    if (r.is_empty()) {
> > branch?
> 
> That would fix
>     p->text()
> but some users may also use
>     p->text(p->page_rect())
> and then they won't get all the text.
> 
> Quite a predicament!

But the documentation doesn't say to use p->page_rect() to use the whole text,
though yes you may be right it may happen, but then we can only say "you're
getting what you asked for", the docu says pass no rectangle to get the whole
text.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/poppler-bugs/attachments/20170731/b05d3f1d/attachment.html>