Skip to content

Update brand logo for Ecosia #23405

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
anthonypkeane opened this issue Jun 10, 2022 · 2 comments · Fixed by brave/brave-core#13745
Closed

Update brand logo for Ecosia #23405

anthonypkeane opened this issue Jun 10, 2022 · 2 comments · Fixed by brave/brave-core#13745
Assignees
Labels

Comments

@anthonypkeane
Copy link

Description

Per https://bravesoftware.slack.com/archives/C015G1THCAJ/p1654884889038839

Please update the brand logo for Ecosia
New Ecosia logo 6 22

@anthonypkeane anthonypkeane added OS/Android Fixes related to Android browser functionality OS/Desktop labels Jun 10, 2022
@simonhong simonhong self-assigned this Jun 13, 2022
simonhong added a commit to brave/brave-core that referenced this issue Jun 13, 2022
@simonhong
Copy link
Member

simonhong commented Jun 13, 2022

It seems desktop doesn't need to update from browser side. Brand logo is updated in backround when user loaded ecosia.com once.
image

@Uni-verse
Copy link
Contributor

Uni-verse commented Jul 1, 2022

Verified on Samsung GS 21 & Tab S7 using

Brave	1.41.86 Chromium: 103.0.5060.66 (Official Build) beta (64-bit) 
Revision	20b1569438a85e631d15e83eb355e3e326e5da6f-refs/branch-heads/5060@{#1066}
OS	Android 12; Build/SP1A.210812.016
  • Verified brand logo update in url bar using new tab page search
  • Verified brand logo is updated in Brave Shields and Brave Rewards for new users
  • Verified brand logo is updated in url, Sheilds, BR panel for upgraded profiles
URL Shields Rewards
screenshot-1656704160776 screenshot-1656704364299 screenshot-1656704377817
screenshot-1656702878961 screenshot-1656704641595 screenshot-1656704653767

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants