Closed
Bug 241363
Opened 21 years ago
Closed 20 years ago
Search box is ugly in Customize Toolbars dialog
Categories
(Firefox :: Toolbars and Customization, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jacobr, Assigned: stdowa+bugzilla)
References
()
Details
(Keywords: fixed-aviary1.0)
Attachments
(2 files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
bugs
:
review+
bugs
:
approval-aviary+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6) Gecko/20040206 Firefox/0.8 StumbleUpon/1.906 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6) Gecko/20040206 Firefox/0.8 When you have set up another search engine in the integrated google search box and then decide to remove the search box by dragging it to the Customize Toolbars dialog you get an ugly looking search box in the Customize Dialog Reproducible: Always Steps to Reproduce: 1.Go To Google Search Box and add another engine (for example IMDb) 2.Go View -> Toolbars -> Customize... and remove the searchbox Actual Results: An ugly looking search box in the Customize Toolbars dialog Expected Results: A beautiful searchbox, perhaps with the standard Google icon
Reporter | ||
Comment 1•21 years ago
|
||
Comment 3•21 years ago
|
||
confirmed linux gtk2 trunk cvs 20040424 firefox, didn't need to add an extra engine to see this
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → All
Comment 4•21 years ago
|
||
this isn't a theme issue actually, seen on multiple themes. I could swear I've seen this before, but that might be on IRC.
Updated•21 years ago
|
Summary: Removing custom-made search boxes gives ugly graphics in Customize Toolbars dialog → Search box is ugly in Customize Toolbars dialog
(In reply to comment #6) > Probably related to bug 250355 Not necessarily, but that may be part of it. The main problem i see is that the search icon/dropmarker thing is actually a XUL button element, which is why it looks like it does in the Customize Toolbars window. It looks "correct" on the toolbar because of the #searchBarDropMarker binding on #search-proxy-button in browser/content/browser/browser.css, which, for some reason, it does not pick up in the palette. I think i attempted to visually fix the problem via the theme a little while back, but couldn't quite get it to work out as well as it did in my own theme. I may give it another whirl, unless someone can truly fix the underlying problem. Mike, I mentioned it before in bug 227745 comment #2, but never filed a bug on it.
Assignee | ||
Comment 8•20 years ago
|
||
The customization dialog doesn't know about the search binding stashed in content/browser.css.
Assignee | ||
Comment 9•20 years ago
|
||
Assignee | ||
Updated•20 years ago
|
Attachment #154073 -
Flags: review?(bugs)
Comment 11•20 years ago
|
||
Comment on attachment 154073 [details] [diff] [review] patch Yeah, OK, although I filed 255836 on what I deem to be bogus dependencies on the browser. This will do for 1.0. Can you make sure it doesn't break thunderbird though first? I'm not sure how the first include isn't...
Attachment #154073 -
Flags: review?(bugs)
Attachment #154073 -
Flags: review+
Attachment #154073 -
Flags: approval-aviary+
Updated•20 years ago
|
Flags: blocking-aviary1.0? → blocking-aviary1.0+
Assignee | ||
Comment 12•20 years ago
|
||
(In reply to comment #11) > (From update of attachment 154073 [details] [diff] [review]) > Yeah, OK, although I filed 255836 on what I deem to be bogus dependencies on > the browser. This will do for 1.0. Can you make sure it doesn't break > thunderbird though first? I'm not sure how the first include isn't... > Firefox is the only app that uses the toolkit/ version, every other app has forked its own.
Updated•20 years ago
|
Assignee: bugs → sdwalker
Whiteboard: [have patch]
Comment 13•20 years ago
|
||
Thunderbird and Sunbird forked their own version because they had to, not because they wanted to. See Bug 250793 as an example. We should fix this centrally to make the forking unnecessary.
Comment 14•20 years ago
|
||
checked in on branch for 1.0, not sure if we want to fix this on trunk or just fix this better instead.
Whiteboard: [have patch] → fixed-aviary1.0
Comment 16•20 years ago
|
||
I think we should resolve this and move any further work over to the more general problem at bug 250355
Comment 17•20 years ago
|
||
marking FIXED, we'll deal with the "right" fix in other bugs.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Keywords: fixed-aviary1.0
Whiteboard: fixed-aviary1.0
Updated•18 years ago
|
QA Contact: bugzilla → toolbars
You need to log in
before you can comment on or make changes to this bug.
Description
•