Skip to content
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 News - publisher card title has incorrect font #13610

Closed
petemill opened this issue Jan 15, 2021 · 1 comment · Fixed by brave/brave-core#9281
Closed

Brave News - publisher card title has incorrect font #13610

petemill opened this issue Jan 15, 2021 · 1 comment · Fixed by brave/brave-core#9281

Comments

@petemill
Copy link
Member

Incorrect

image
image

Correct

image

Incorrect as of 1.21.2

@petemill petemill self-assigned this Jan 15, 2021
@rebron rebron added the feature/brave-news formerly brave-today label Apr 19, 2021
@bsclifton bsclifton changed the title [Desktop] Brave Today - publisher card title has incorrect font [Desktop] Brave News - publisher card title has incorrect font Jun 14, 2021
@petemill petemill assigned nullhook and unassigned petemill Jun 29, 2021
@nullhook nullhook added this to the 1.28.x - Nightly milestone Jul 2, 2021
@stephendonner
Copy link

Verified FIXED using

Brave 1.28.74 Chromium: 92.0.4515.93 (Official Build) nightly (x86_64)
Revision 6eb43ff7850a1d710c3f827a0555737c74edab5c-refs/branch-heads/4515@{#1378}
OS macOS Version 11.4 (Build 20F71)

Steps:

  1. new profile
  2. launched Brave
  3. opened a new-tab page
  4. scrolled down
  5. clicked on Show Brave News
  6. scrolled down through content and took note of fonts with protruding stems (like y, g, etc.)
example example
Screen Shot 2021-07-12 at 3 02 08 PM Screen Shot 2021-07-12 at 3 02 38 PM

@rebron rebron changed the title [Desktop] Brave News - publisher card title has incorrect font Brave News - publisher card title has incorrect font Jul 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants