[web] add icons, reply and chevron small
ClosedPublic
Actions

Authored by • benschac on Feb 10 2022, 11:40 AM.

Details

Reviewers

atul
ashoat

Commits

rCOMM3b7e9222ae6b: [web] add icons, reply and chevron small

Summary

screenshot, add these two

Image 2022-02-10 at 2.43.09 PM.jpg (878×2 px, 565 KB)

Test Plan

n/a make sure icons in the app don't break

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• benschac created this revision.Feb 10 2022, 11:40 AM

Herald added subscribers: • karol, • adrian, tomek, ashoat. · View Herald TranscriptFeb 10 2022, 11:40 AM

add appIcons.json

• benschac edited the summary of this revision. (Show Details)Feb 10 2022, 11:44 AM

Harbormaster completed remote builds in B6728: Diff 9544.Feb 10 2022, 11:45 AM

Harbormaster completed remote builds in B6729: Diff 9545.Feb 10 2022, 11:48 AM

• benschac requested review of this revision.Feb 10 2022, 11:48 AM

• benschac added a reviewer: ashoat.Feb 10 2022, 6:01 PM

Please explain the need for the urgency in short-circuiting the typical review process

This revision is now accepted and ready to land.Feb 10 2022, 8:54 PM

In D3167#84249, @ashoat wrote:

Please explain the need for the urgency in short-circuiting the typical review process

Chatted about it IRL. Atul is mostly OOO traveling today. It would be okay to short-circuit the review process.

Closed by commit rCOMM3b7e9222ae6b: [web] add icons, reply and chevron small (authored by • benschac). · Explain WhyFeb 11 2022, 7:05 AM

This revision was automatically updated to reflect the committed changes.

• benschac added a commit: rCOMM3b7e9222ae6b: [web] add icons, reply and chevron small.

Revision Contents
Changeset List

Path

Size

web/

icons/

AppIcons.json

163 lines

selection.json

180 lines

Diff 9567

View Options

web/icons/AppIcons.json