Page MenuHomePhabricator

[web] Remove fontsource
ClosedPublic

Authored by ashoat on Mar 8 2022, 4:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 5, 12:51 AM
Unknown Object (File)
Oct 16 2024, 3:52 AM
Unknown Object (File)
Oct 16 2024, 3:52 AM
Unknown Object (File)
Oct 16 2024, 3:52 AM
Unknown Object (File)
Oct 16 2024, 3:52 AM
Unknown Object (File)
Oct 7 2024, 6:57 AM
Unknown Object (File)
Oct 7 2024, 6:57 AM
Unknown Object (File)
Oct 7 2024, 6:57 AM

Details

Summary

See context in ENG-848 – on prod it's not working, and it's causing serious performance effects on the Node server backend as well as causing the website to take 15s to load.

As of D3122 (which "fixed" ENG-675), the fonts are getting loaded from Google Fonts anyways. These lines aren't working, so there should be no negative impact to removing them.

Test Plan

Best I can do is test yarn prod in my local environment. From investigating the prod web environment with Chrome Dev Tools, I'm pretty sure this is a safe change

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/remove-fontsource
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul published this revision for review.Mar 8 2022, 4:10 PM
atul accepted this revision.
This revision is now accepted and ready to land.Mar 8 2022, 4:11 PM
This revision was automatically updated to reflect the committed changes.