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

fix(ui): more font optimization #1284

Merged
merged 4 commits into from
Dec 18, 2024
Merged

fix(ui): more font optimization #1284

merged 4 commits into from
Dec 18, 2024

Conversation

peps
Copy link
Collaborator

@peps peps commented Dec 18, 2024

This PR further optimizes the font situation within the app. This should improve performance and bundle size slightly.

  • Removed DrukWide 500 and Poppins 800 since they were not used in many places
  • Converted Druk fonts to woff2 format. This makes a huge difference in performance and file size.
  • Remove Google Fonts and bundled IBM Plex Mono with the app
  • Deleted all unused fonts from the /public folder ( I think they were still getting bundled, but not loaded )

@peps peps requested a review from PanchoBubble December 18, 2024 09:01
@peps peps self-assigned this Dec 18, 2024
@peps peps requested review from brianp and stringhandler December 18, 2024 09:01
@peps peps marked this pull request as ready for review December 18, 2024 09:02
@brianp brianp merged commit bc085e9 into tari-project:main Dec 18, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PanchoBubble PanchoBubble Awaiting requested review from PanchoBubble

@brianp brianp Awaiting requested review from brianp

@stringhandler stringhandler Awaiting requested review from stringhandler

Assignees

@peps peps

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@peps @brianp