1669297 - Update Bing logo to new icon (and possibly name)

Status: VERIFIED FIXED

(Firefox :: Search, task, P2)

Description

[Mike Kaply [:mkaply]]

Bing released a new logo and renamed Bing to Microsoft Bing:

https://www.theverge.com/2020/10/5/21502315/microsoft-bing-rebrand-search-engine-logo

We should update to the new logo and decide if the name change is necessary.

Comment 1

[Mark Banner (:standard8)]

Changing the name is currently likely to loose the user's settings relating to Bing. Unfortunately we still index on name.

Comment 2

[Mike de Boer [:mikedeboer]]

We could hacker-de-hack it in the UI...

Comment 4

[Mark Banner (:standard8)]

Mike & I looked at Bing's logo standards and we believe the name is ok as it currently is.

Comment 5

[Mark Banner (:standard8)]

Attachment: Bug 1669297 - Update Bing's logo for the search engine. r?mkaply!.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dfdf69fc9e42
Update Bing's logo for the search engine. r=mkaply.

Comment 7

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/dfdf69fc9e42

Comment 8

[Cristian Baica [:cbaica], Release Desktop QA]

Verified on Windows 10, macOS 10.12 and Ubuntu 18.04 using the latest Fx85.0a1.

Verified multiple points where the bing logo is displayed:

• URL bar
• enabled search bar
• websearch bar
• default search engine drop-down
• search shortcut section (removed and re-added to verify nothing broke)

Comment 9

[Mike Kaply [:mkaply]]

Comment on attachment 9189184 [details]
Bug 1669297 - Update Bing's logo for the search engine. r?mkaply!.

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: Branding issue with Bing
• User impact if declined: Wrong bing logo
• Fix Landed on Version: 85
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Just an icon change
• String or UUID changes made by this patch:

Comment 10

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9189184 [details]
Bug 1669297 - Update Bing's logo for the search engine. r?mkaply!.

Approved for 78.7esr.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-esr78/rev/bb0aec0abc2d

Comment 12

[Ryan VanderMeulen [:RyanVM]]

ESR-only test follow-up:
https://hg.mozilla.org/releases/mozilla-esr78/rev/76dd55244af0

Comment 13

[Florencia Di Ciocco, NI to vbalducci]

Attachment: bing addon.png

Hi,

I downloaded the Beta ES-Ar version, and for Argentina bing is not a default search engine. So I added it manually. The addon had the new icon but the one-off did not have the new icon. You can also see that the icon of the addon is right on the upper right corner.

Comment 14

[Florencia Di Ciocco, NI to vbalducci]

Attachment: Snag_b92d00a.png

It also has the old icon on about:preferences#search

Comment 15

[Mark Banner (:standard8)]

The Bing add-on points to https://www.bing.com/favicon.ico as the icon to use for their search engine definition. We don't have any control over their add-on, that is separate to the application defined engine.

To test this, you need to use a locale/region where Bing is defined by Firefox.

Comment 16

[Florencia Di Ciocco, NI to vbalducci]

Attachment: esr

Hi Mark, sorry for answering so late, I forgot to add myself to cc so I didn't get a notification when you answered.

So I tested this on the esr en-us build and I still see the old logo. Could you check, please?

Thanks, Flor.

Comment 17

[Mark Banner (:standard8)]

According to comment 10, I think you need 78.7esr - that looks like 78.6.1esr.

Comment 18

[Florencia Di Ciocco, NI to vbalducci]

Oh thanks for the tip. So 78.7 isn't available yet, so I'll test this again in about 3 weeks.

Comment 19

[Florencia Di Ciocco, NI to vbalducci]

Attachment: top sites

Hi Mark, so I tested this on esr 78.7, it looks good except that the topsite icon is showing the old one. This also happens on Nightly. I didn't check beta or release. Is this a bug on our side or is bing sending the wrong picture?
Thanks, Flor.

Comment 20

[Mark Banner (:standard8)]

Thanks for pointing that out - we missed it (it is wrong in central as well). I've filed bug 1688088 to fix that.

Comment 21

[Florencia Di Ciocco, NI to vbalducci]

You're welcome. Since there's a new bug for topsites, I'll mark esr as verified.
Thanks Mark for all of the responses.