The default bug view has changed. See this FAQ.

No search icon or text on global search bar

RESOLVED FIXED in Thunderbird 3.1b2

Status

Thunderbird
Search
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: standard8, Unassigned)

Tracking

({regression})

Trunk
Thunderbird 3.1b2
regression
Dependency tree / graph

Firefox Tracking Flags

(blocking-thunderbird3.1 beta2+, thunderbird3.1 beta2-fixed)

Details

Attachments

(2 attachments)

(Reporter)

Description

7 years ago
When bug 545955 landed it had completely broken the mac build. We found this to be due to the inclusion of textbox.css into extraCustomizeItems.xul, which NeilAway pointed out may be a bad thing as that's designed to be included in xbl files anyway (unless he was talking about the content version).

In any case, removing it fixed the mac issues, but broke the icons and text on the global search bar on Windows and Linux (i.e. broke = not displayed). As this seemed the better of two evils, we've left the textbox.css removed from the file for now and this bug is for fixing up the search icon and text.
Created attachment 440736 [details]
screenshot of Linux behaviour
Yeah, I added that to get the search icon to show up.  The autocomplete widget lacks such niceties, and I wanted to keep search.xml as simple as possible, so I added the button back in extraCustomizeItems.xml instead of in the XBL content for search.xml.  My assumption was that accordingly the styling had to come from the document since it would not be part of the anonymous sub-content, but maybe the XBL CSS will affect the children?  We also can move stuff back into search.xml...
I pushed a bustage-fix for this, testing the patch on all 3 platforms first:
http://hg.mozilla.org/comm-central/rev/9759a8e7417f
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
status-thunderbird3.1: --- → beta2-fixed
Target Milestone: --- → Thunderbird 3.1b2

Comment 4

7 years ago
Can you attach the patch also to this bug, so that other people can review it, and include the necessary changes also in their themes?

Comment 5

7 years ago
Created attachment 442008 [details] [diff] [review]
Use gtk-find for gnomestripe

Same as bug 525971.
Attachment #442008 - Flags: review?(mkmelin+mozilla)

Comment 6

7 years ago
Comment on attachment 442008 [details] [diff] [review]
Use gtk-find for gnomestripe

Thx for the patch. r=mkmelin
(Would have been better to put the patch in a follow-up bug though)
Attachment #442008 - Flags: review?(mkmelin+mozilla) → review+

Comment 7

7 years ago
(In reply to comment #6)
Thank you for reviewing and a comment.
I'll do so next time I found a bug which is a follow-up one.

Could you add "checkin-needed" to keywords?
I have no permission to check in.
(Reporter)

Comment 8

7 years ago
(In reply to comment #7)
> (In reply to comment #6)
> Thank you for reviewing and a comment.
> I'll do so next time I found a bug which is a follow-up one.
> 
> Could you add "checkin-needed" to keywords?
> I have no permission to check in.

No, we're going to need to move this to a follow-up bug. As this bug is already marked fixed for beta 2 and we've already shipped that. Otherwise we're not going to be easily able to track this getting fixed.

As you didn't know about that, I'll file a follow-up bug and transfer the patch there in a few mins.

Comment 9

7 years ago
OK.
I'll file new bug for comment #5 shortly.

Comment 10

7 years ago
(In reply to comment #9)
File as bug 563647.
(Reporter)

Updated

7 years ago
Depends on: 563654

Updated

7 years ago
Depends on: 563647

Comment 11

7 years ago
_updateVisibleText()?
I think this method is gone in comm-central (See Bug Bug 547224).

I got this error.


       -- Exception object --
       + message (string) 'this._updateVisibleText is not a function'
       + fileName (string) 'chrome://messenger/content/search.xml'
       + lineNumber (number) 140
       + stack (string) '()@chrome://messenger/content/search.xml:140
       '
       + name (string) 'TypeError'
       *
       -- Stack Trace --
       ()@chrome://messenger/content/search.xml:140
(In reply to comment #11)
> _updateVisibleText()?
> I think this method is gone in comm-central (See Bug Bug 547224).

Yes, this is gone from mozilla-central and so affects comm-central but not comm-1.9.2.  Please file a new bug for it if it does not already exist.

Updated

7 years ago
Depends on: 566282
You need to log in before you can comment on or make changes to this bug.