311041 - search/find ignores <br>

Status: NEW

(Toolkit :: Find Toolbar, defect, P5)

Description

[Wolfgang Rosenauer [:wolfiR]]

I will attach a very small testcase which shows that a search in the web-page
(in that case with search word 'afasel' will show a wrong result.

Comment 1

[Wolfgang Rosenauer [:wolfiR]]

Attachment: small testcase

Comment 2

[Adam Guthrie]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051004
Firefox/1.6a1 ID:2005100411

Could you please provide a bit more details? Expected results, actual results
and whatnot? I'm really not sure what you expect to happen here. For me,
searching for "afasel" highlights the word across the line break and /does/ find
the word.

Comment 3

[Wolfgang Rosenauer [:wolfiR]]

sorry. Yes, the word "afasel" is found. But I don't expect that because afasel
is no word in this document.
There are two words "bla" and "fasel" so "afasel" doesn't exist. It shouldn't
find anything.

Comment 4

[O. Atsushi (Torisugari)]

IMHO, this is an invalid/wontfix request.

At least it's not a "Search" bug. -> Find Toolbar or Find Fast?

Comment 5

[Adam Guthrie]

Well, we do not do the same thing for new lines, and since a <br> is technically
a new line...

Comment 6

[Uri Bernstein]

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

Comment 7

[Johannes Buchner]

Did anyone notice that if you write "fasel" in the search and go to the start of the input and write "bl", nothing is found. so far, so good. when you complete the word with an "a" to "blafasel", the word is found and marked, but the search box stays red, claiming that it started on top. Bug or feature? 

Anyhow, not that important.

Comment 9

[Martijn Wargers (dead)]

Attachment: patch

This fixes it, but I haven't really tested it much.

Comment 10

[Martijn Wargers (dead)]

Attachment: small testcase2

There are also search issues with <button> and I suspect there are more elements where you wouldn't want to search in.

Comment 11

[Martijn Wargers (dead)]

(In reply to comment #10)
> There are also search issues with <button> and I suspect there are more
> elements where you wouldn't want to search in.

Not issues that the patch is introducing.

Comment 12

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.