931481 - Poor text selection behaviour in PDFs

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, defect, P3)

Description

[Clemens Eisserer]

Attachment: selection_nofocus.png

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Firefox/24.0 (Beta/Release)
Build ID: 20130910160258

Steps to reproduce:

Loaded http://www.stronachinstitut.at/wp-content/uploads/2012/02/Nachbaur_CV5.pdf with FireFox' internal PDF viewer.

This also happens with pdf.js git.


Actual results:

1. For the mentioned PDF the selection rectangle drawn by Firefox doesn't match the actual text.

2. Another issue which can be seen with this PDF is that selection boxes sometimes overlap, leading to horizonal lines (when the window has focus)

3. When the window is out-of-fox, selected text is drawn with wrong y coordinate (as shown in screenshot)

Comment 1

[[:Aleksej]]

Reproduced with 2013-10-28-03-02-05-mozilla-central-firefox-27.0a1.en-US.linux-x86_64.  Spaces are not visibly selected, but there are small areas where selection goes the wrong way, as if there was "nothing" there.

Comment 2

[Petruta Horea [:phorea], Desktop Test Engineering]

Mozilla/5.0 (X11; Linux x86_64; rv:27.0) Gecko/20100101 Firefox/27.0
Reproduced with latest Aurora (20131118004001).

Comment 3

[(mostly gone) XtC4UaLL [:xtc4uall]]

Also confirmed against Mozilla/5.0 (Windows NT 6.1; WOW64; rv:30.0) Gecko/20100101 Firefox/30.0 ID:20140213030201 CSet: a62bde1d6efe

"PDF 7f1922abe9e8c7428dd448211df1b1b [1.5 Adobe PDF Library 10.0 / Acrobat PDFMaker 10.1 for Word] (PDF.js: 0.8.649)"
"page=1 - getOperatorList: time=%dms, len=63"
"text indexing: page=1 - time=%dms"

Comment 4

[Calixte Denizet (:calixte)]

A lot of improvements have been made in the last years so this bug is likely fixed.
If it's still reproducible, please reopen and attach the pdf to the bug report.

Comment 5

[Marco Castelluccio [:marco]]

It's much better than before, but still partially buggy.

Comment 6

[Calixte Denizet (:calixte)]

We found the pdf on web.archive.org, and it's still buggy:
https://web.archive.org/web/20141210054406/http://www.stronachinstitut.at/wp-content/uploads/2012/02/Nachbaur_CV5.pdf

Comment 7

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/pdf.js/pull/14266

Comment 8

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub master push: https://github.com/mozilla/pdf.js/commit/28fb3975eb762bc616ef4ee7d74d5633ab786bc2

Commit merged into master by GitHub
Authored by Jonas Jenwald (Snuffleupagus)