Closed
Bug 337173
Opened 20 years ago
Closed 20 years ago
When removing the active search engine, gray search engine text doesn't get updated
Categories
(Firefox :: Search, defect)
Tracking
()
RESOLVED
WORKSFORME
Firefox 2 beta1
People
(Reporter: mozilla, Assigned: mozilla)
References
Details
Attachments
(2 files)
|
2.97 KB,
patch
|
Gavin
:
review+
mconnor
:
superreview+
mconnor
:
approval-branch-1.8.1+
|
Details | Diff | Splinter Review |
|
4.72 KB,
image/png
|
Details |
| Assignee | ||
Comment 1•20 years ago
|
||
Attachment #221343 -
Flags: review?(gavin.sharp)
| Assignee | ||
Comment 2•20 years ago
|
||
Comment 3•20 years ago
|
||
Comment on attachment 221343 [details] [diff] [review]
Fixes problem; also adds a small refactoring so that tests for the gray text state are more legible
Apparently this is only an issue with Linux, so I haven't reproduced the bug, but this looks good either way.
Attachment #221343 -
Flags: review?(gavin.sharp) → review+
| Assignee | ||
Updated•20 years ago
|
Attachment #221343 -
Flags: superreview?(mconnor)
| Assignee | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Flags: blocking-firefox2?
Whiteboard: swag:.5d
Target Milestone: Firefox 2 alpha2 → Firefox 2 beta1
Updated•20 years ago
|
Flags: blocking-firefox2? → blocking-firefox2+
Updated•20 years ago
|
Attachment #221343 -
Flags: superreview?(mconnor)
Attachment #221343 -
Flags: superreview+
Attachment #221343 -
Flags: approval-branch-1.8.1+
| Assignee | ||
Comment 4•20 years ago
|
||
I think this problem must've gotten fixed during one of the iterations. I can no longer reproduce this on Linux, which is where I was seeing it before. Resolving, if anyone can repro in the future we'll reopen.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → WORKSFORME
Whiteboard: swag:.5d
Updated•19 years ago
|
Flags: blocking-firefox2+
You need to log in
before you can comment on or make changes to this bug.
Description
•