275040 - find toolbar highlight + :first-letter: sequence of letters is mixed

Status: RESOLVED FIXED

(Toolkit :: Find Toolbar, defect)

Description

[Gyuris Gellért]

The highlight function of the Find toolbar mixes the sequence of some
letters on the page if there is a ":first-letter" selector. CSS definition
itself is not necessary, the selector is enough.

Load the first attachment and type a letter "a" into the find toolbar and
click the highlight button. As a result the sequence of the letters will be
wrong.

Comment 1

[Gyuris Gellért]

Attachment: testcase

Comment 2

[Nickolay_Ponomarev]

heh, note: there's a missing '</head>'. If I add it or if I remove the doctype,
everything works as it should. Still, something is broken here.

Comment 3

[Gyuris Gellért]

Attachment: testcase2

Sorry. Repairing the html file the bug is still present.

Comment 4

[Nickolay_Ponomarev]

Attachment: correct testcase

Your second attachment didn't have a close tag for <meta> (should be <meta
.../> in xhtml). Also style rules are applied to H1 and P (notice the case). I
attached the correct testcase. It indeed breaks for me, so there's a problem in
FT code.

Comment 5

[Boris Zbarsky [:bzbarsky]]

The patch in bug probably 277686 fixes this (can't tell, since find toolbar is
broken in current debug builds).

Comment 6

[Boris Zbarsky [:bzbarsky]]

Fixed by checkin for bug 277686

Comment 7

[Slyoldfox]

From:
http://forums.mozillazine.org/viewtopic.php?t=212215

"Cold, take a look at "Most popular headlines" and then do it. It's right under
the flash based advertisement to the right of the page. Go to
http://news.com.com/ and type the letter "a" in the find toolbar and click the
highlight button. As a result SOME words get literally "split" up now.

Can someone either file a new bug or reopen the one I previously mentioned? I
don't have access. Sad

BUILD: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b) Gecko/20050202
Firefox/1.0+
"

I am actually getting this with 1.0 official release:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0

Should this be REOPENED ?

Comment 8

[Uri Bernstein]

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

Comment 9

[Martijn Ras]

I'm reopening this bug as this still happens in Firefox 1.0.5. I close bug
#301334, which i just opened ... after tweaking the CSS file a bit i narrowed
the error down to a couple of elements and properties that might be the cause
and found this bug.

Comment 10

[Martijn Ras]

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

Comment 11

[Jaime Mitchell (use [e-mail] for email)]

(In reply to comment #9)
> I'm reopening this bug as this still happens in Firefox 1.0.5. 

This was fixed on the trunk as per comment 6. Therefore restoring resolution. 

Please reopen only if you can reproduce this with a current trunk build (such as
deer park alpha 2). The branch builds have only had security fixes for the past
6 months.