Text selection is extended when searching for copied text in pdf
Categories
(Firefox :: PDF Viewer, defect)
Tracking
()
People
(Reporter: pmagyari, Unassigned)
Details
Attachments
(1 file)
(deleted),
image/png
|
Details |
Steps to reproduce:
- Open Firefox
- Navigate to https://unec.edu.az/application/uploads/2014/12/pdf-sample.pdf
- Open the find-in-text search bar (ctrl+f/ cmd+f)
- Highlight and copy the following phrase from the first sentence of the text "all of the fonts"
- Paste it into the search bar
Expected results:
The phrase is searched for and it is highlighted on the page, the text selection from earlier remains the same ("all of the fonts")
Actual results:
The phrase is highlighted but the text-selection is extended (see screenshot)
Note
This only occurs when selecting a phrase that spans across multiple rows
Comment 1•3 years ago
|
||
Calixte, could you take a look? It's likely a regression of https://github.com/mozilla/pdf.js/pull/13261.
Peter, can you reproduce this with other PDFs too or just the one you attached?
Reporter | ||
Comment 2•3 years ago
|
||
Yes, I can reproduce this on every PDF I've checked.
Reporter | ||
Comment 3•3 years ago
|
||
Sorry, I didn't mean to remove the NI from Calixte.
Updated•3 years ago
|
Comment 4•3 years ago
|
||
Set release status flags based on info from the regressing bug 1754585
Updated•3 years ago
|
Comment 5•3 years ago
|
||
I'm not convinced that this is actually a regression, since with a slight modification of the STR this can also be reproduced in e.g. the current Firefox ESR version as well.
Steps to reproduce:
- Open in Firefox ESR
- Navigate to https://unec.edu.az/application/uploads/2014/12/pdf-sample.pdf
- Highlight the following phrase from the first sentence of the text "all of the fonts"
- Open the find-in-text search bar (ctrl+f/cmd+f) and paste "allof the fonts" (note the missing space) in the find-input
Doing this also reproduces this bug, which as far as I'm concerned rules out this being a regression from PR https://github.com/mozilla/pdf.js/pull/13261 and this rather seem to be a much older issue.
Comment 6•3 years ago
|
||
Given comment 5, this is not a regression.
Comment 7•3 years ago
|
||
(In reply to Marco Castelluccio [:marco] from comment #6)
Given comment 5, this is not a regression.
At least, not a regression from https://github.com/mozilla/pdf.js/pull/13261.
Peter, with the modified STR, could you check if this is an older regression?
Reporter | ||
Comment 8•3 years ago
|
||
I was able to reproduce it without the space (Comment 5) on a build from 2019-10-03 so it is indeed not a recent regression but for some reason I have an issue opening builds older than that, so I wasn't able to run mozregression properly.
Description
•