Closed Bug 947275 Opened 10 years ago Closed 10 years ago

Update Bing favicon to match current branding

Categories

(Firefox :: Search, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 29
Tracking Status
firefox27 --- wontfix
firefox28 --- fixed
firefox29 --- fixed
b2g-v1.3 --- fixed

People

(Reporter: mconnor, Assigned: mconnor)

References

Details

(Whiteboard: [qa-])

Attachments

(1 file, 1 obsolete file)

Attached patch bingIconUS (obsolete) — Splinter Review
For both stock and Metro.  Using the yellow version of their icons to match bing.com.
Attachment #8343786 - Flags: review?(jaws)
Attachment #8343786 - Attachment is patch: true
Comment on attachment 8343786 [details] [diff] [review]
bingIconUS

Review of attachment 8343786 [details] [diff] [review]:
-----------------------------------------------------------------

We should have HiDPI icons for Bing on desktop. See bug 795495 for how this was done for Google.
Attachment #8343786 - Flags: review?(jaws) → review-
Attachment #8343786 - Flags: review- → review+
Attached patch bingIconUSSplinter Review
Attachment #8343786 - Attachment is obsolete: true
Attachment #8343945 - Flags: review?(gavin.sharp)
Attachment #8343945 - Flags: review?(gavin.sharp) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/8396f173b5b7
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Bugs get marked as fixed when they reach mozilla-central.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/8396f173b5b7
Status: ASSIGNED → RESOLVED
Closed: 10 years ago10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
Depends on: 951465
Comment on attachment 8343945 [details] [diff] [review]
bingIconUS

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: 
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky): 
String or IDL/UUID changes made by this patch:

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: outdated/mismatched icon, especially noticeable on launch of Metro
Testing completed (on m-c, etc.): has been baking since December, it's on Aurora already
Risk to taking this patch (and alternatives if risky): almost nil
String or IDL/UUID changes made by this patch: none
Attachment #8343945 - Flags: approval-mozilla-beta?
Attachment #8343945 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.