-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Brave Together text and image assets should read Brave Talk, in new-tab page's cards #16798
Milestone
Comments
Here's the link to the updated asset in Figma: https://www.figma.com/file/0pOzvvueMoeaCxKaVoU59Z/Widget-discovery?node-id=2506%3A23053 :) |
bsclifton
added a commit
to brave/brave-core
that referenced
this issue
Jul 26, 2021
Updates widget itself, settings screen (under customize), and a few comments in the code Fixes brave/brave-browser#16798
24 tasks
Verified
Steps:
|
@bsclifton mind getting an uplift going for |
7 tasks
Changing label to |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Brave Together text and image assets should read Brave Talk, in new-tab page's cards
Steps to Reproduce
Customize
on the bottom rightCards
Brave Together
's text and image+ Add
...
and look at the resulting menu itemActual result:
Cards
paneBrave Together
widgetHide Brave Together
menu itemExpected result:
Should read
Brave Talk
in each of the above 3 UI elements.Reproduces how often:
100%
Brave version (brave://version info)
@karenkliu can you/the team help provide new assets? Thanks!
/cc @simonhong @petemill (I'll leave renaming to the pros, since I don't know what else I could break with simple text changes!)
The text was updated successfully, but these errors were encountered: