294615 - Search/find fails on words containing soft hyphen (­)

Status: RESOLVED FIXED

(Core Graveyard :: Embedding: APIs, defect, P3)

Description

[Mikko Virkkilä]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.6) Gecko/20050306 Firefox/1.0.1 (Debian package 1.0.1-2)
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.6) Gecko/20050306 Firefox/1.0.1 (Debian package 1.0.1-2)

Soft hyphens (­) are never rendered in Firefox (Bug 9101). Many sites (esp
news outlets) use ­ since every other browser has support for them. This
causes problems for find, since "dis­cre­tion­a­ry" will be
rendered as "discretionary" but will not mach when you search for it.

Reproducible: Always

Steps to Reproduce:
1. Go to: https://bugzilla.mozilla.org/attachment.cgi?id=906
2. Press ctrl+f
3. Type "discretionary"
Actual Results:  
The word is not found

Expected Results:  
The ­ should be ignored, and the word should be found.

Comment 1

[Jonathan Haas]

Bug 9101 is fixed, but this bug is still present.

We should ignore ­ when searching, if possible.

Comment 2

[Jesse Ruderman]

http://www.w3.org/TR/REC-html40/struct/text.html#h-9.3.3 says "For operations such as searching and sorting, the soft hyphen should always be ignored."

Comment 3

[Asaf Romano (gone)]

Attachment: patch

Comment 4

[Asaf Romano (gone)]

mozilla/embedding/components/find/src/nsFind.cpp 1.47

Comment 5

[Jonathan Haas]

We _may_ also want to ignore ​ (zero width space), which can be used as a line-breaking character without visible hyphen (for example in URLs).

Comment 6

[Martijn Wargers (dead)]

Hi Jonathan, that seems a good idea to me, I filed bug 389651 for it.