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.

Sign up for GitHub

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

Jump to bottom

Font diff between notification action and content #3494

Closed
nikclayton opened this issue Mar 29, 2023 · 3 comments · Fixed by #3663
Closed

Font diff between notification action and content #3494

nikclayton opened this issue Mar 29, 2023 · 3 comments · Fixed by #3663

Comments

@nikclayton
Copy link
Contributor

Screenshot_20230329-134749.png

Per this screenshot there's a font diff between the first line, the notification action, and subsequent lines, the content.

So it looks weird that the username, repeated on both lines, is displayed differently.
@connyduck
Copy link
Collaborator

What device is this from?

@nikclayton
Copy link
Contributor Author

Pixel 4a

@nikclayton
Copy link
Contributor Author

I see this on a physical device, but not an emulator.

Some checking shows it's the @android:style/TextAppearance.DeviceDefault.Small in item_status_notification.xml. That doesn't appear to be used in any other layout, so can be safely deleted.

nikclayton added a commit that referenced this issue May 16, 2023
Remove android:textAppearance="@android:style/TextAppearance.DeviceDe…
c02db5d 
…fault.Small"

It caused text size differences between the text in this view and all the other textviews in this layout.

It's not used in other layouts.

Fixes #3494
nikclayton added a commit that referenced this issue Jun 11, 2023
Remove android:textAppearance="@android:style/TextAppearance.DeviceDe…
327254d 
…fault.Small" (#3663)

It caused text size differences between the text in this view and all the other textviews in this layout.

It's not used in other layouts.

Fixes #3494
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove android:textAppearance="@android:style/TextAppearance.DeviceDefault.Small" tuskyapp/Tusky
2 participants
@nikclayton @connyduck