Page MenuHomePhabricator

[yarn] Restore `fortawesome`-related dependencies to fix icon issue
ClosedPublic

Authored by atul on Jun 8 2022, 7:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jan 30, 5:35 PM
Unknown Object (File)
Thu, Jan 30, 5:35 PM
Unknown Object (File)
Thu, Jan 30, 9:02 AM
Unknown Object (File)
Sun, Jan 26, 3:29 PM
Unknown Object (File)
Jan 9 2025, 5:43 AM
Unknown Object (File)
Jan 9 2025, 3:58 AM
Unknown Object (File)
Jan 7 2025, 6:11 PM
Unknown Object (File)
Jan 7 2025, 12:47 PM

Details

Summary

We were experiencing strange behavior with fortawesome icons. They were either not appearing correctly (way too large) or not appearing at all.

Linear context:
https://linear.app/comm/issue/ENG-1228/filters-icon-way-too-large-on-web-calendar

I think the fortawesome/react-fontawesome dependency needs to be bumped in lockstep with its peer dependencies in order to keep things working.

With this change I just rolled back and fixed version numbers so we're at a state where things look as expected on both dev and prod.

I'll separately look at bumping these dependencies while preserving the appearance. (Might be a place to try snapshot testing to prevent this sort of regression in the future?)

Test Plan

Tested on dev and prod and things looked as they used to:

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable