Closed Bug 373371 Opened 16 years ago Closed 14 years ago

Findbar highlighting shows background-image


(Toolkit :: Find Toolbar, defect)

Not set





(Reporter: u279076, Unassigned)



(Keywords: testcase, Whiteboard: [fixed by bug 263683])


(3 files, 1 obsolete file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv: Gecko/20070219 Firefox/
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv: Gecko/20070219 Firefox/

When doing a page search for "firefox" and selecting "highlight all", the highlighting on firefox in an image rendered the image as the highlighting.

Reproducible: Always

Steps to Reproduce:
1. navigate to
2. press Ctrl+F
3. type in "firefox" into the find box
4. click "next"
5. click "Highlight all"

Actual Results:  
The background highlighting is the image-behind being rendered instead of the highlighting colour

Expected Results:  
The text should simply be highlighted without background image.

See screenshot:

This has been reported on WinXP and Linux using FF1.5,, and Mac.
Ever confirmed: true
confirmed for windows, mac, linux, i see the same as anthony
This is because of CSS in file /style/rustico/content.css on Solution of this is to add the following lines to file.

#main-feature span span,
#main-feature span span,
#main-feature .brief-feature span span
  background-image: none;

Sorry for my English.
My next solution is shorter, but it depends on id defined in file on lines 276 and 311.

#main-feature #__firefox-findbar-search-id {
  background-image: none;
Sorry for confirming it without thinking, it should be a duplicate of bug 263683, IMO. :)
duplicate per #4
Closed: 16 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 263683
Sorry, but I can't create a patch file, then I post file with descriptions of changes on branches 1.8 and 1.8.0.

I will learn working with CVS soon. ;)
Attached patch Patch for 1.8 Branch (obsolete) — Splinter Review
Attachment #258087 - Attachment is obsolete: true
Component: General → Find Toolbar / FastFind
QA Contact: general → fast.find
Reopening. This bug is about one of the symptoms of the findbar highlighting, showing a background-image were it shouldn't.

Bug 263683 is about reworking the findbar highlighting by using selections instead of manipulating the DOM (adding <span>s all over the place). Fixing bug 263683 would probably fix this bug as well, but it's not the same bug.

Marek, the 1.8 branch is mainly for security fixes. On the trunk, the code has been refactored into findbar.xml. You might want to patch this:
Severity: normal → minor
Depends on: 263683
Resolution: DUPLICATE → ---
Summary: Find Highlighting rendering image-behind instead of highlighting → Findbar highlighting shows background-image
Version: unspecified → Trunk
Product: Firefox → Toolkit
The design on was changed. So the testcase doesn't work anymore or I'm not able to see a similar issue. Anyone, who can give us a new testcase?
Whiteboard: [testcase needed]
Attached file testcase
This is working correctly now that bug 263683 is fixed.
Closed: 16 years ago14 years ago
Keywords: testcase
Resolution: --- → FIXED
Whiteboard: [testcase needed]
Thanks for the testcase, Martijn; verified fixed using, with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:1.9.1a2pre) Gecko/2008080504 Minefield/3.1a2pre
Should we have the testcase in the test-suite or at least in Litmus?
Flags: in-testsuite?
Flags: in-litmus?
Per discussion with Martijn, he will create an automated test (reftest) for that issue.
Flags: in-litmus? → in-litmus-
Whiteboard: [fixed by bug 263683]
Target Milestone: --- → mozilla1.9.1a2
With a little tweak of the reftest in bug 263683, this bug could be tested too.
But the reftest still depends on bug 448676, since it needs to have enhanced privileges.
(In reply to comment #16)
> With a little tweak of the reftest in bug 263683, this bug could be tested too.
> But the reftest still depends on bug 448676, since it needs to have enhanced
> privileges.

In that case we should add the bug to the depends list to get a notification when its possible.
Depends on: 448676
Sorry, I was wrong, bug 448676 might be invalid/wontfix. It probably more depends on bug bug 374458 or bug 430652, see bug 263683, comment 53.
Mmh, so we better should keep track of it in bug 263683?
No longer depends on: 448676
You need to log in before you can comment on or make changes to this bug.