Closed
Bug 1030622
Opened 10 years ago
Closed 6 years ago
Second search searches initial text with new hightlighting
Categories
(Firefox :: PDF Viewer, defect, P3)
Tracking
()
VERIFIED
FIXED
Firefox 65
People
(Reporter: pepijndevos, Assigned: Snuffleupagus)
References
Details
(Keywords: regression, ux-consistency, Whiteboard: [pdfjs-c-ff-integration])
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:30.0) Gecko/20100101 Firefox/30.0 (Beta/Release)
Build ID: 20140605174243
Steps to reproduce:
Open https://wiki.hexonet.net/images/0/01/DOMAIN_API_Reference.pdf
ctrl+f domain
select a word
ctrl+c ctrl+f
Actual results:
Pressing ctrl+g goes to the next occurrence of domain, with a highlight that is as long as the selected word.
Expected results:
It should have searched for the selected word.
Reporter | ||
Updated•10 years ago
|
Component: Untriaged → PDF Viewer
Reporter | ||
Updated•10 years ago
|
OS: Mac OS X → All
Hardware: x86 → All
Comment 1•10 years ago
|
||
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:33.0) Gecko/20100101 Firefox/33.0
Reproduced on latest Nightly 20140624030200, set as New.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 2•10 years ago
|
||
Regression window(m-c)
Good:
http://hg.mozilla.org/mozilla-central/rev/a76c1f4c4112
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:19.0) Gecko/19.0 Firefox/19.0 ID:20121017110913
Bad:
http://hg.mozilla.org/mozilla-central/rev/5142bbd4da12
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:19.0) Gecko/19.0 Firefox/19.0 ID:20121017191029
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=a76c1f4c4112&tochange=5142bbd4da12
Regression window(m-c)
Good:
http://hg.mozilla.org/integration/mozilla-inbound/rev/e78db739d589
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:19.0) Gecko/19.0 Firefox/19.0 ID:20121017121312
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/d896a7f47e02
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:19.0) Gecko/19.0 Firefox/19.0 ID:20121017123513
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=e78db739d589&tochange=d896a7f47e02
Regressed by:
af98d67916ad Ryan VanderMeulen — Bug 801280 - Update pdf.js to version 0.6.39. r=dtownsend
Comment 3•10 years ago
|
||
I'm sorry I cannot reproduce the issue with the STR in the comment 0. Please provide more detailed instructions. The regression windows does not look valid as well -- the bug adds support for PDF find support (before that viewer could only search only pages that are visible). Based on initially specified FF version 30 and vague STR, I can assume that we are talking about bug 1006714 (which will be fixed in FF32).
Flags: needinfo?(pepijndevos)
Version: 19 Branch → 30 Branch
Comment 4•10 years ago
|
||
This bug happened since Firefox19.
And reproduced on latest Nightly.
https://hg.mozilla.org/mozilla-central/rev/de8c0f0e74a2
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:34.0) Gecko/20100101 Firefox/34.0 ID:20140728030201
Steps to reproduce:
1. Open https://wiki.hexonet.net/images/0/01/DOMAIN_API_Reference.pdf
2. Ctrl+f
3. Type 'domain' without quotation
--- 'DOMAIN' is highlighted as expected
4. Select a word (e.g. API) by mouse drag
5. Ctrl+c to copy selected text to clipboard
6. Ctrl+f
--- Now, clipboard text('API') copied into the find field
7. Ctrl+G to find next
Actual Results
Firefox trying to find "dom" instead the find field text "api"
Expected Results
Firefox should find "api" which is current text in the find field
Comment 5•10 years ago
|
||
Okay, I was trying to replicate the same steps on Mac OSX -- find bar does not work the same way there. Cmd+F shall just pull the previous search term.
Flags: needinfo?(pepijndevos)
Updated•10 years ago
|
Keywords: ux-consistency
Comment 6•10 years ago
|
||
TESTING
Linux Mint 17
Firefox for Ubuntu, 34.0a1 (2014-07-29)
Confirmed by following steps in Comment 4
Updated•10 years ago
|
Priority: -- → P3
Whiteboard: [pdfjs-c-ff-integration]
Assignee | ||
Updated•6 years ago
|
Status: NEW → RESOLVED
Closed: 6 years ago
Depends on: 1506596
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
Updated•6 years ago
|
Assignee: nobody → jonas.jenwald
status-firefox63:
--- → wontfix
status-firefox64:
--- → wontfix
status-firefox65:
--- → fixed
status-firefox-esr60:
--- → wontfix
Updated•6 years ago
|
Flags: qe-verify+
Comment 7•6 years ago
|
||
I have managed to reproduce this issue using Firefox 62.0a1 (BuildId:20180606220131).
This issue is verified fixed using Firefox 65.0b9 (BuildId:20190107180200) on Windows 10 64bit and Ubuntu 16.04 64bit.
You need to log in
before you can comment on or make changes to this bug.
Description
•