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
F3378653: D3391.diff
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
Unknown Object (File)
Oct 22 2024, 11:15 AM
Unknown Object (File)
Oct 22 2024, 11:15 AM

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
Branch
update-icons-last-time
Lint
No Lint Coverage
Unit
No Test Coverage

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.