Skip to content

content: Use 17 for base font size (and height 22), like in Figma #541

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
Merged
merged 1 commit into from
Mar 1, 2024
Merged

content: Use 17 for base font size (and height 22), like in Figma #541

merged 1 commit into from
Mar 1, 2024

Conversation

chrisbobbe
Copy link
Collaborator

@chrisbobbe chrisbobbe commented Feb 29, 2024 •
edited by gnprice
Loading

This is an alternative to #540 that doesn't include a fix for #538. Screenshots coming soon.

Fixes: #512

@chrisbobbe chrisbobbe added the a-content Parsing and rendering Zulip HTML content, notably message contents label Feb 29, 2024
@chrisbobbe
Copy link
Collaborator Author

Before After
image image
image image
image image
image image

@gnprice
Copy link
Member

gnprice commented Mar 1, 2024

Thanks! Looks good; merging.

@gnprice gnprice force-pushed the pr-font-size-17-only branch from 699de36 to 3bcfe2c Compare March 1, 2024 00:11
@gnprice gnprice merged commit 3bcfe2c into zulip:main Mar 1, 2024
@gnprice gnprice mentioned this pull request Mar 1, 2024
Closed
@chrisbobbe chrisbobbe mentioned this pull request Mar 1, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
a-content Parsing and rendering Zulip HTML content, notably message contents
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Increase content font size
2 participants
@chrisbobbe @gnprice