Update DuckDuckGo icons to match new branding
Categories
(Firefox :: Search, task, P2)
Tracking
()
People
(Reporter: csasca, Assigned: jkt)
References
Details
(Whiteboard: [sng-])
Attachments
(2 files, 1 obsolete file)
57.08 KB,
image/png
|
Details | |
48 bytes,
text/x-phabricator-request
|
diannaS
:
approval-mozilla-beta+
RyanVM
:
approval-mozilla-esr115+
|
Details | Review |
Found in
- Firefox 115.0b1
Affected versions
- Firefox 115.0b1
- Firefox 116.0a1
Tested platforms
- Affected platforms: macOS 12.6.6, Ubuntu 22.04, Windows 11
Preconditions
- Have DuckDuckGo set as the default search engine
Steps to reproduce
- Launch Firefox
- Access duckduckgo.com
- Focus the address bar and type anything
Expected result
- The DuckDuckGo search icon is identical to the one used on the DuckDuckGo webpage
Actual result
- The DuckDuckGo search icon is slightly different than the one used on the DuckDuckGo webpage
Regression range
- Seems that on builds from 2022 the same icon is used as in present and an old icon is used on 2020 builds.
Additional notes
- The difference can be seen in the attachment
Comment 1•2 years ago
|
||
:csasca, if you think that's a regression, could you try to find a regression range using for example mozregression?
Comment 2•2 years ago
|
||
Are you comparing the fact that the one on the tab looks slightly fatter and the feather touches the circle? If so, then I think you are seeing the lower resolution icon.
They have several icons listed within their page that Firefox could be using, I think you are seeing this one: https://duckduckgo.com/favicon.ico
There are also these ones:
- https://duckduckgo.com/assets/icons/meta/DDG-iOS-icon_76x76.png
- https://duckduckgo.com/assets/icons/meta/DDG-iOS-icon_120x120.png
- https://duckduckgo.com/assets/icons/meta/DDG-iOS-icon_152x152.png
Their main page also uses this icon: https://duckduckgo.com/assets/logo_homepage_mobile.normal.v109.svg
I think these icons are most like the one we use for the search engine, so I think this is a wontfix, unless I'm missing something?
Reporter | ||
Comment 3•2 years ago
|
||
Yep, the one on the tab has the circle radius lower and is more bolded, while for the other circle the radius is larger and estompated.
Comment 4•2 years ago
|
||
Given that the image that Firefox uses are closer to the ones on the main DDG page (and to me it looks clearer than the one on the tab), I think it would be best to stay with those unless DDG raise an issue.
Assignee | ||
Comment 5•2 years ago
|
||
Indeed that is DDG's new logo, I have a patch ready and will attach here.
Reopening, let me know if I should re-raise.
Comment hidden (obsolete) |
Assignee | ||
Comment 7•2 years ago
|
||
Assignee | ||
Updated•2 years ago
|
Comment 8•2 years ago
|
||
(In reply to Jonathan Kingston [:jkt] he/him from comment #5)
Indeed that is DDG's new logo, I have a patch ready and will attach here.
I can't see anything in their brand guidelines about smaller sized icons, and as I mentioned previously I think the one we have is clearer that their favicon.
This was last updated in bug 1731538 / bug 1730070, and I don't see any difference in the main icon since then.
Please can you explain what makes you think this is a new logo? We can always go back to them and check, but I'd like to understand what is driving this.
Assignee | ||
Comment 9•2 years ago
|
||
I'm currently employed by DDG and found this task as part of our internal one to update the branding. I have sent you an email to confirm this.
The logo is subtly different, the beak has changed shape (most notable the lower part is wider at the thinnest part), the neck is a little wider around the beak and there is a bolder circle.
Comment 10•2 years ago
|
||
Thank you for the extra info, I wasn't aware of that.
Comment 11•2 years ago
|
||
Comment 12•2 years ago
|
||
bugherder |
Comment 13•2 years ago
|
||
Comment on attachment 9342125 [details]
Bug 1836958 - Update DuckDuckGo favicon with latest version. r?Standard8
Beta/Release Uplift Approval Request
- User impact if declined: DuckDuckGo icon will be out of date.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Icon-only change.
- String changes made/needed: None
- Is Android affected?: No
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: Update branding of DuckDuckGo for the long lived ESR branch.
- User impact if declined: N/A
- Fix Landed on Version:
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Icon-only change.
Comment 14•2 years ago
|
||
Comment on attachment 9342125 [details]
Bug 1836958 - Update DuckDuckGo favicon with latest version. r?Standard8
Approved for 116.0b2
Comment 15•2 years ago
|
||
bugherder uplift |
Updated•2 years ago
|
Comment 16•2 years ago
|
||
Comment on attachment 9342125 [details]
Bug 1836958 - Update DuckDuckGo favicon with latest version. r?Standard8
Approved for 115.1esr.
Comment 17•2 years ago
|
||
uplift |
Comment 18•2 years ago
|
||
bugherder uplift |
Updated•2 years ago
|
Comment 19•2 years ago
|
||
This issue is Verified as fixed in our latest 115.1.0esr Beta 116.0b3 and our latest Nightly build.
Description
•