Closed Bug 1268605 Opened 8 years ago Closed 8 years ago

Quick filter icon doesn't look like a funnel

Categories

(Thunderbird :: Theme, defect)

45 Branch
defect
Not set
trivial

Tracking

(thunderbird46 wontfix, thunderbird47 fixed, thunderbird48 fixed, thunderbird49 fixed, thunderbird_esr4546+ fixed)

RESOLVED FIXED
Thunderbird 49.0
Tracking Status
thunderbird46 --- wontfix
thunderbird47 --- fixed
thunderbird48 --- fixed
thunderbird49 --- fixed
thunderbird_esr45 46+ fixed

People

(Reporter: pete_a90, Assigned: Paenglab)

Details

Attachments

(3 files)

Thunderbird 45 changed the quickfilter icon from a magnifying glass to a funnel but it doesn't even look like a funnel. The nozzle end is fat and short.
Severity: normal → trivial
Attached patch 1268605.patchSplinter Review
Yes, the nozzle was a bit fat on Win8+ and OS X 10.10+
The other platforms and inverted stay as they are. They look correct.
Assignee: nobody → richard.marti
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #8747165 - Flags: review?(aleth)
Screenshot for easier review.
Will it now find messages faster, less diameter means faster flow, right ;-)
[Sorry, I hope the Swiss have more humour than the Germans]
Attachment #8747165 - Flags: review?(aleth) → review+
(In reply to Jorg K (GMT+2) from comment #3)
> Will it now find messages faster, less diameter means faster flow, right ;-)

Of course, icon changes have a big influence on program functionality. :-)
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/ddff26e23765b8afe9157ec6a2745a33022c2d6d
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Keywords: checkin-needed
OS: Unspecified → All
Hardware: Unspecified → All
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 49.0
Do you want uplift for this? Uplift is later today or on Sunday.
Flags: needinfo?(richard.marti)
Comment on attachment 8747165 [details] [diff] [review]
1268605.patch

Yes, I was already on filling the request. ;-)

[Approval Request Comment]
User impact if declined: not so nice QFB icon
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): no risk
Flags: needinfo?(richard.marti)
Attachment #8747165 - Flags: approval-comm-esr45?
Attachment #8747165 - Flags: approval-comm-beta?
Attachment #8747165 - Flags: approval-comm-aurora?
Attachment #8747165 - Flags: approval-comm-aurora? → approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: