Port bug 773641 to Thunderbird's WebSearch.

RESOLVED FIXED in Thunderbird 18.0

Status

Product:
Thunderbird
Component:
Search
Status:
RESOLVED FIXED
Reported:
6 years ago
Modified:
6 years ago

People

(Reporter: bwinton, Assigned: Paenglab)

Tracking

Version:
unspecified
Target:
Thunderbird 18.0
Platform:
x86
Mac OS X
Bug Flags:
in-testsuite -

Thunderbird Tracking Flags

(thunderbird17 fixed)

Details

Attachments

(1 attachment)

patch
5.47 KB, patch
bwinton
: review+
standard8
: approval-comm-aurora+
Details | Diff | Splinter Review
(Reporter)

Description

•
6 years ago 
Or just use the icon that's landing in central, if we can do that.
(Assignee)

Comment 1

•
6 years ago 
Created attachment 656173 [details] [diff] [review]
patch

Using the Twitter icon we have already for chat.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #656173 - Flags: review?(bwinton)
(Reporter)

Comment 2

•
6 years ago 
Comment on attachment 656173 [details] [diff] [review]
patch

I'm going to be annoyed if mozilla-central ends up with a different icon, but for now, r=me.  ;)

(Also, what do you think about base64-ing the other search icons?)
Attachment #656173 - Flags: review?(bwinton) → review+
(Reporter)

Comment 3

•
6 years ago 
As a note, please don't land this until we have the final permission to use these icons in the dependent bug…

Thanks,
Blake.
(Assignee)

Comment 4

•
6 years ago 
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #2)
> Comment on attachment 656173 [details] [diff] [review]
> patch
> 
> I'm going to be annoyed if mozilla-central ends up with a different icon,
> but for now, r=me.  ;)

M-c has now also a patch and is using the same icon. The only difference is, I optimized the PNG image to be smaller.

> (Also, what do you think about base64-ing the other search icons?)

All other icons except the AOL icon are also base64 encoded. I don't know how to convert this to base64 (I can't create a png of it). But I also see, http://search.aol.com/aol/search uses now a other icon. Should I use this?
(Reporter)

Comment 5

•
6 years ago 
Nah, let's leave it the way it is.
(Assignee)

Comment 6

•
6 years ago 
The related bug landed. Is it okay land this too?
(Reporter)

Comment 7

•
6 years ago 
Yep.
 (Assignee)

Updated

•
6 years ago
Keywords: checkin-needed

Comment 8

•
6 years ago 
https://hg.mozilla.org/comm-central/rev/d66004eb6054
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 18.0
 (Assignee)

Comment 9

•
6 years ago 
Comment on attachment 656173 [details] [diff] [review]
patch

[Approval Request Comment]
As it's not clear if and when TB 18 and ff. will be released as a normal version, it would be good if this new icon can land on TB 17 and TB 17 ESR.
Attachment #656173 - Flags: approval-mozilla-aurora?

Comment 10

•
6 years ago 
Comment on attachment 656173 [details] [diff] [review]
patch

I think you mean to ask for comm-aurora approval here, changing the flags.
Attachment #656173 - Flags: approval-mozilla-aurora? → approval-comm-aurora?
Attachment #656173 - Flags: approval-comm-aurora? → approval-comm-aurora+

Comment 11

•
6 years ago 
Checked in: https://hg.mozilla.org/releases/comm-aurora/rev/ff17ebcb7403
status-thunderbird17: --- → fixed
You need to log in before you can comment on or make changes to this bug.