Make use of firefox search icon

RESOLVED FIXED in Thunderbird 3.0b3

Status

Thunderbird
General
--
enhancement
RESOLVED FIXED
9 years ago
8 years ago

People

(Reporter: andreasn, Assigned: clarkbw)

Tracking

Trunk
Thunderbird 3.0b3
All
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(6 attachments, 5 obsolete attachments)

(Reporter)

Description

9 years ago
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.7) Gecko/2009030422 Ubuntu/8.10 (intrepid) Firefox/3.0.7 Ubiquity/0.1.5
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b4pre) Gecko/20090318 Shredder/3.0b3pre

Firefox have a much nicer search icon than Thunderbird currently.

Reproducible: Always
(Reporter)

Comment 1

9 years ago
Created attachment 368230 [details] [diff] [review]
patch against nightly

As the icon is already shipped inside global, all that's needed is to change the url.

Updated

9 years ago
Severity: normal → enhancement
Status: UNCONFIRMED → NEW
Ever confirmed: true
Version: unspecified → Trunk
Comment on attachment 368230 [details] [diff] [review]
patch against nightly

Not sure who this should go to.
Attachment #368230 - Flags: review?
(Assignee)

Comment 3

9 years ago
Cool, this looks super easy. :)  I'll make a patch out of this on Saturday.  Maybe I can bump another review over to philor
Assignee: nobody → clarkbw
Status: NEW → ASSIGNED
Hardware: x86 → All
Target Milestone: --- → Thunderbird 3.0b3
(Assignee)

Comment 4

8 years ago
Created attachment 370100 [details] [diff] [review]
hg patch using ff search icon

here's the patch that changes us over to using the FF search icon.  I noticed that the drop down arrow intersects the new search icon because it is at a different angle than ours currently is.  Not sure if that's intended, will attach screenshot.
Attachment #368230 - Attachment is obsolete: true
Attachment #368230 - Flags: review?
(Assignee)

Comment 5

8 years ago
Created attachment 370102 [details] [diff] [review]
version 2, which includes the necessary jar.nm file change

missed including the jar.nm file
Attachment #370100 - Attachment is obsolete: true
(Assignee)

Comment 6

8 years ago
Created attachment 370103 [details]
screenshot of search icons compared

here's the comparison of the search icons TB2 vs. TB3 + previous patch
(Assignee)

Comment 7

8 years ago
Created attachment 370107 [details]
search box w/ arrow on left side

andreas: i tried this alternate version where I took the gif, auto-cropped it, and converted it to a png.  Then changed this css in the searchBox.css

.quick-search-button-dropmarker {
  margin-top: 14px;
  list-style-image: url("chrome://messenger/skin/icons/search-arrow.png");
}

If that's something we want to do we instead, we can change the patch to that.  

Hopefully we can get philor to review it once we've got a patch that's ready.
What happens if you just get rid of (or decrease, I don't know if some is required to not have it clear at the top) the margin-top on the dropmarker, which is shoving it down toward the bottom? Isn't the natural and normal spot for a dropmarker vertically centered, and it was only being pushed down because we were using a Windows-style leaning-right magnifying glass that needed the dropmarker shoved down below its glass?

Are there native apps that use the (I presume that's native direction) leaning-right magnifying glass with a dropmarker, and if so where do they put it?
(Assignee)

Comment 9

8 years ago
Created attachment 370178 [details]
screenshot of philor's suggestion in action

(In reply to comment #8)
> What happens if you just get rid of (or decrease, I don't know if some is
> required to not have it clear at the top) the margin-top on the dropmarker,
> which is shoving it down toward the bottom? Isn't the natural and normal spot
> for a dropmarker vertically centered, and it was only being pushed down because
> we were using a Windows-style leaning-right magnifying glass that needed the
> dropmarker shoved down below its glass?

It's amazing how simple that is.  Here's the screenshot showing what that looks like.  I only bumped the margin-start up 1px so it doesn't overrun the search icon.

> Are there native apps that use the (I presume that's native direction)
> leaning-right magnifying glass with a dropmarker, and if so where do they put
> it?

I couldn't find a single other use of our next generation search icon w/ drop marker, the future must not be evenly distributed.
(Reporter)

Comment 10

8 years ago
Bryan: that looks great!

Couldn't find any search icons with drop downs either. Evolution, Banshee and Transmission all use search icons that pops up a menu when you click them, but none of them use any arrow.
Created attachment 370179 [details] [diff] [review]
middle aligned new dropdown arrow against the new search

here's the patch that makes it happen.

updated jar.mn file
removed the old Search-bar icon
converted search-arrow.gif to search-arrow.png
css changes to make it all happen
Attachment #370102 - Attachment is obsolete: true
(Assignee)

Updated

8 years ago
Attachment #370179 - Flags: review?(philringnalda)
How close to working in RTL is Gnomestripe's searchbox? Can we drop in bug 481860's -moz-transform: scaleX(-1) to flip the image, and get everything happy?
Created attachment 370741 [details]
screenshot of the RTL quick search

(In reply to comment #12)
> How close to working in RTL is Gnomestripe's searchbox? Can we drop in bug
> 481860's -moz-transform: scaleX(-1) to flip the image, and get everything
> happy?

Got it working.  Had to figure out how to test in the RTL mode.  Ended up writing a blog entry on it. http://clarkbw.net/blog/2009/04/02/testing-rtl-in-thunderbird/

Will attach patch for review in a sec.
Created attachment 370743 [details] [diff] [review]
updated patch w/ RTL fix

here's the updated patch for review.  Only real change is the addition of the chromedir="rtl" selector.  I could have done a more specific child selector, but it doesn't look very pretty.

#quick-search-button[chromedir="rtl"] > stack > hbox > .quick-search-button-image
Attachment #370179 - Attachment is obsolete: true
Attachment #370743 - Flags: review?(philringnalda)
Attachment #370179 - Flags: review?(philringnalda)
(Assignee)

Updated

8 years ago
Whiteboard: [waiting on review philor]
Attachment #370743 - Flags: review?(philringnalda) → review+
Comment on attachment 370743 [details] [diff] [review]
updated patch w/ RTL fix

Sorry to be so slow - you've picked up a bit of rot in the jar.mn hunk, but other than that it looks great.
Created attachment 374134 [details] [diff] [review]
[checked in] update w/ fuzz fixes

(In reply to comment #15)
> Sorry to be so slow - you've picked up a bit of rot in the jar.mn hunk, but
> other than that it looks great.

Thanks Phil!  Updated to fix the rot issues.  carrying forward the review+
Attachment #370743 - Attachment is obsolete: true
Attachment #374134 - Flags: review+
I should probably continue in here or spin out other bugs for qute and pinstripe themes to do the same thing.
Keywords: checkin-needed
Whiteboard: [waiting on review philor]
Comment on attachment 374134 [details] [diff] [review]
[checked in] update w/ fuzz fixes

Checked in: http://hg.mozilla.org/comm-central/rev/69b81aa3618c

I'll let Bryan decide where he wants to do pinstripe and qute.
Attachment #374134 - Attachment description: update w/ fuzz fixes → [checked in] update w/ fuzz fixes
Keywords: checkin-needed
(Assignee)

Updated

8 years ago
Whiteboard: [needs patch for pinstripe and qute]
(Reporter)

Comment 19

8 years ago
Filed bug and patch for Vista (and XP) icon: https://bugzilla.mozilla.org/show_bug.cgi?id=506626

Need to check the status on pinstripe.
After that I think we can close this.
(Reporter)

Comment 20

8 years ago
Created attachment 391070 [details]
screenshot of the search icon in pinstripe

Here is a screenshot of Thunderbird on OS X. Firefox main window at the top, Thunderbird in the middle and Firefox bookmarks in the bottom. The icons in FX bookmarks and TB looks the same.
(Reporter)

Updated

8 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
I fixed Mac in bug 491360.
Depends on: 506626
Whiteboard: [needs patch for pinstripe and qute]
You need to log in before you can comment on or make changes to this bug.