Paths

Table of Contentst

[lib/native] add user-filled to comm icons pack
ClosedPublic
Actions

Authored by ginsu on Jun 23 2023, 11:04 AM.

Details

Reviewers

atul
rohan

Commits

rCOMM6119ef78e56f: [lib/native] add user-filled to comm icons pack

Summary

This diff adds the user-filled icon to the comm icons pack. These changes were copy and pasted from files given to me by @ted

I also added the icons to our dropbox

Test Plan

Please see the screenshot below:

Screenshot 2023-06-23 at 11.02.50 AM.png (136×196 px, 8 KB)

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Jun 23 2023, 11:04 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJun 23 2023, 11:04 AM

ginsu edited the summary of this revision. (Show Details)Jun 23 2023, 11:05 AM

ginsu edited the test plan for this revision. (Show Details)

ginsu added reviewers: atul, rohan.

ginsu attached a referenced file: F604099: Screenshot 2023-06-23 at 11.02.50 AM.png. (Show Details)

ginsu published this revision for review.Jun 23 2023, 11:08 AM

Harbormaster completed remote builds in B20473: Diff 28055.Jun 23 2023, 11:36 AM

Thanks! Icon looks good, I just don't know how icon file changes work so adding @atul as blocking

Was this based off of comm-icon-config.json from the repo?

It looks like there are changes unrelated to the new icon being added.

that said, accepting to unblock

lib/shared/comm-icon-config.json
409–425 ↗	(On Diff #28055)	It looks like there are unrelated changes?

This revision is now accepted and ready to land.Jun 26 2023, 7:22 AM

Was this based off of comm-icon-config.json from the repo?

The comm-icon-config.json was copied and pasted from the selection.json in this folder from dropbox

cc @ted

The comm-icon-config.json was copied and pasted from the selection.json in this folder from dropbox

Shouldn't it be based on what's in the repo?

This revision now requires changes to proceed.Jun 26 2023, 1:08 PM

address comments

Sweet! Could you check web as well before landing?

This revision is now accepted and ready to land.Jun 27 2023, 12:42 PM

Harbormaster completed remote builds in B20556: Diff 28190.Jun 27 2023, 12:55 PM

add icons CommIcons type on web

ginsu added inline comments.Jun 27 2023, 2:05 PM

web/CommIcon.react.js
20–27 ↗	(On Diff #28192)	Forgot to add these icon names to this type when I previously added the new icons, double checked each of these and made sure they look as expected

This revision was landed with ongoing or failed builds.Jun 27 2023, 2:07 PM

Closed by commit rCOMM6119ef78e56f: [lib/native] add user-filled to comm icons pack (authored by ginsu). · Explain Why

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMM6119ef78e56f: [lib/native] add user-filled to comm icons pack.

Harbormaster completed remote builds in B20558: Diff 28192.Jun 27 2023, 2:23 PM

rohan mentioned this in D8467: [lib/native/web] add user edit icon to comm icon pack.Jul 10 2023, 1:41 PM

Revision Contents
Changeset List

Path

Size

lib/

shared/

comm-icon-config.json

142 lines

native/

fonts/

CommIcons.ttf

Diff 28190

View Options

lib/shared/comm-icon-config.json