1282759 - Support whole word search for find in page

Status: VERIFIED FIXED

(Firefox :: PDF Viewer, defect, P3)

Description

[Mike de Boer [:mikedeboer]]

Since bug 269442, the find toolbar support a new mode, called 'Whole Words', which only matches strings surrounded by word-breaking characters, like spaces or punctuation marks in latin-derived languages.

As you might know, this is the default matching mode for the find toolbars in Webkit/ Blink based browsers.

The implementation uses the term 'entireword', not 'wholeword' as you might think. This is due to historical reasons that are not very interesting for this bug ;-)

Comment 3

[Jonas Jenwald [:Snuffleupagus]]

Attachment: entireWord.patch

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e6d4af6ebdd5
Include the `entireWord` mode when forwarding find events to the PDF.js viewer. r=bdahl

Comment 5

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/e6d4af6ebdd5

Comment 6

[Daniel Bodea [:danibodea]]

Hi Mike and Jonas,
Can one of you provide some steps to reproduce for this issue?
Thanks!

Comment 7

[Mike de Boer [:mikedeboer]]

Ah, I think Jonas should be the one to get you an STR here - if he's available, otherwise Brendan could. My guess here:
Open up a PDF document containing text, open up the find-in-page bar, toggle the 'While Words' option button and start trying to find whole words in the document.
The ER should be that only whole words can be found, not partial matches.

Comment 8

[Daniel Bodea [:danibodea]]

I have reproduced the issue in Nightly v64.0a1 from 2018-09-05 and I have verified it in Nightly v64.0a1 from 2018-10-19.
I have also verified if it occurs in firefox63 and firefox62 and set the tags accordingly but considering that the target was set as Firefox 64, I have set the general status of this bug as VERIFIED.

The steps taken to verify the issue are the following:
1. Open the browser and reach: http://etd.lib.byu.edu/PDFCreation/EditingTextinPDFDocuments.pdf
2. Tap: "CTRL+F" to use the search in page feature;
3. Click the "Whole words" button from the find in page bar to activate this option.
4. Search for the word "you" and make sure that the word "your" is not highlighted as a result.
Affected: The word "your" is also highlighted as a result.
Fixed: The word "your" is not highlighted as a result.

Thank you.