Find toolbar doesn't reset status when search is wrapped

RESOLVED FIXED in mozilla1.9alpha2

Status

()

defect
P3
minor
RESOLVED FIXED
14 years ago
10 years ago

People

(Reporter: gaubugzilla, Assigned: mano)

Tracking

Trunk
mozilla1.9alpha2
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Reporter

Description

14 years ago
How to reproduce this problem:
1. Go to https://bugzilla.mozilla.org/
2. Press Ctrl-F to open search toolbar
3. enter "test" - the word "latest" gets highlighted
4. now move to the beginning of the search field and add the letters "la"

Result: after adding the second letter the word "latest" is found again, the text "reached end of text, continued from top" is displayed, but the input field stays red. It should be white instead, as is the case when the letters "la" are added in reverse order.

Seeing this with in:
Windows NT 5.1; en-US; rv:1.8.0.1) Gecko/20060111 Firefox/1.5.0.1
Windows NT 5.1; en-US; rv:1.8) Gecko/20060317 Firefox/2.0a1
Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060309 Firefox/1.6a1
Posted patch patchSplinter Review
Assignee: nobody → mano
Status: NEW → ASSIGNED
Attachment #247008 - Flags: review?(gavin.sharp)
Priority: -- → P3
Target Milestone: --- → Firefox 3 alpha2
Attachment #247008 - Flags: review?(gavin.sharp) → review+
mozilla/toolkit/content/widgets/findbar.xml 1.3
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
OS: Windows XP → All
Hardware: PC → All
Resolution: --- → FIXED
tbd: unit test.
Flags: in-testsuite?
Posted file unit test (obsolete) —
Attachment #247582 - Flags: review?(gavin.sharp)
Comment on attachment 247582 [details]
unit test

nit: space after "after" in both ASSERT strings.
Attachment #247582 - Flags: review?(gavin.sharp) → review+
Reporter

Comment 6

13 years ago
Gavin, you probably meant <before "after"> :)
mozilla/toolkit/content/tests/chrome/Makefile.in 1.4
mozilla/toolkit/content/tests/chrome/bug331215_window.xul initial revision: 1.1
mozilla/toolkit/content/tests/chrome/test_bug331215.xul initial revision: 1.1
Flags: in-testsuite? → in-testsuite+

Updated

11 years ago
Blocks: 443493
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.