Closed Bug 347961 Opened 18 years ago Closed 18 years ago

Search engine button doesn't change for autodetected engine

Categories

(Firefox :: Search, defect)

Product:
Firefox
Component:
Search
Version:
2.0 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 2 beta2

People

(Reporter: pamg.bugs, Assigned: pamg.bugs)

References

Details

(Keywords: fixed1.8.1, Whiteboard: [rolled into 347400])

The pinstripe theme doesn't make the search-engine button glow or otherwise indicate the presence of an autodetected engine on a page.
Flags: blocking-firefox2?
Hmm, I'm not seeing any special "glow" with a windows branch build, either (testcase is http://technorati.com/).
Joe was handling all the glow detection work but I believe he's on vacation now. Pam - any chance you can take a look?
Joe did the winstripe glow work. It looks like his patch https://bugzilla.mozilla.org/attachment.cgi?id=232037&action=view for bug 345466 was never merged into the other searchbar.css changes. I don't think he looked at pinstripe at all. I'll take a look at both themes' detection and display of autodetected engines after I finish with bug 347400, if nobody else has gotten to them first.
OS: Mac OS X 10.4 → All
Hardware: Macintosh → All
Summary: Mac theme doesn't change search engine button for autodetected engine → Search engine button doesn't change for autodetected engine
Blocks: 345407
Working on this now. I'll roll the changes into the upcoming patch for bug 347400, since they affect many of the same CSS files.
Assignee: jgoldman → pamg.bugs
Flags: blocking-firefox2? → blocking-firefox2+
A patch has been submitted for bug 347400 that includes fixes for this problem too.
Whiteboard: [rolled into 347400]
Keywords: fixed1.8.1
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.