98730 - Find/Replace not finding a string with more than one space

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[TucsonTester2]

From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 5.5; Windows 95)
BuildID:    2001090303

Find/Replace will not find a string with more than one space 

Reproducible: Always
Steps to Reproduce:
1.Open a normal document in Editor 
2.Enter a string with more than one space(e.g. "he  llo") in the Find dialog 
that also exists in document 
3.Verify that you're able to find all occurences of that string with the spaces 
in it. 


Actual Results:  The message returned is "The text you entered was not found"

Expected Results:  I expected to see the string I entered into find/replace 
highlighted in the document.

Find/Replace will find any and all occurrences of a string with only one space 
but, 2 or more spaces in the string will not be found.

Comment 1

[Kathleen :Brade]

-->sfraser
Do we need a component for Find dialog?  Who owns Find now?

I'm guessing this issue is (a) Find not detecting the spaces due to where it is 
looking (b) extra spaces are actually nbsp's (have to have <pre> text to confirm) 
or (c) both.  

I would expect that the Find dialog should be smarter about matching spaces with 
nbsp's (do we need a special checkbox for this?).

Comment 2

[Simon Fraser [no longer active]]

I guess the Find code should consider nbsp's and spaces to be equivalent.

Comment 3

[Christopher Aillon (sabbatical, not receiving bugmail)]

*** Bug 103539 has been marked as a duplicate of this bug. ***

Comment 4

[kinmoz]

--> kin

I just posted a patch in bug 103539 that reworks the whitespace ignoring code in
find ... it also fixes this bug.

Comment 5

[kinmoz]

Fix checked in on trunk:

    mozilla/editor/txtsvc/src/nsFindAndReplace.cpp  revision 1.8

Comment 6

[michael wendell]

Verified.