Page MenuHomePhabricator

[web] Introduce `CommIcon` component
ClosedPublic

Authored by atul on Aug 16 2022, 12:56 PM.
Tags
None
Referenced Files
F3687409: D4850.id15683.diff
Tue, Jan 7, 12:32 AM
F3682365: D4850.diff
Mon, Jan 6, 6:17 PM
Unknown Object (File)
Tue, Dec 31, 9:17 AM
Unknown Object (File)
Sun, Dec 29, 8:24 PM
Unknown Object (File)
Sun, Dec 29, 8:24 PM
Unknown Object (File)
Wed, Dec 11, 4:33 PM
Unknown Object (File)
Dec 4 2024, 8:49 PM
Unknown Object (File)
Nov 29 2024, 11:11 AM
Subscribers

Details

Summary

Use the same lib/shared/comm-icon-config.json (exported from IcoMoon) as we're using for CommIcon on native. This ensures that CommIcon on native and web are synced up. Before the icons from IcoMoon were exported separately on native and web which led to some inconsistency.

Heavily inspired by the SWMansionIcon.react.js component in the same directory.

Test Plan

Was able to consume CommIcon component as expected. Tested implicitly by subsequent diffs that replace some usages of SWMansionIcon with CommIcon.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable