Page MenuHomePhabricator

[web] Remove fontsource
ClosedPublic

Authored by ashoat on Mar 8 2022, 4:08 PM.
Tags
None
Referenced Files
F3298395: D3372.diff
Sun, Nov 17, 6:20 AM
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

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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.