Page MenuHomePhabricator

[web] [chore] add remaining icons to the project icon component
ClosedPublic

Authored by benschac on Mar 9 2022, 2:35 PM.
Tags
None
Referenced Files
F3393074: D3391.id10253.diff
Sat, Nov 30, 11:21 AM
Unknown Object (File)
Wed, Nov 27, 1:59 PM
Unknown Object (File)
Wed, Nov 27, 12:09 PM
Unknown Object (File)
Fri, Nov 22, 2:20 PM
Unknown Object (File)
Fri, Nov 22, 2:20 PM
Unknown Object (File)
Fri, Nov 22, 2:20 PM
Unknown Object (File)
Fri, Nov 22, 2:20 PM
Unknown Object (File)
Fri, Nov 8, 10:39 PM

Details

Summary

Add additional icons to the project. I looked through all of the web figma files and these we're the remaining icons that had not been imported.

Image 2022-03-09 at 5.35.34 PM.jpg (1×3 px, 355 KB)

(white icon is the reply bubble which I had to manually export out of Figma because it wasn't in the SWM icon pack)

Test Plan

Icon component should still work as expected. It will support additional icons that have not been imported (icomoon json svg json comming in the next diff)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btw can we use the icons that already exist in lib/shared/swmansion-icon-config.json?

This revision is now accepted and ready to land.Mar 9 2022, 9:07 PM
In D3391#91577, @atul wrote:

btw can we use the icons that already exist in lib/shared/swmansion-icon-config.json?

Not sure, I'd imagine we'd need to rename the icons we have already set here though, some of these icons didn't have names. Additionally, I removed the prefix outline or duotone.