[web] [remove] [ENG-530] remove arrow from inline reply
AbandonedPublic
Actions

Authored by • benschac on Apr 29 2022, 11:09 AM.

Details

Reviewers

atul
tomek

Summary

Remove arrow and associated styles for arrows in inline reply functionality. Note, I grepped the project and confirmed these styles aren't used in other parts on the product.

Image 2022-04-29 at 2.11.43 PM.jpg (208×386 px, 7 KB)

figma file:
https://www.figma.com/file/a1nkbWgbgjRlrOY9LVurTz/Comm-%2F-Desktop-app?node-id=1170%3A78019

There's no longer an arrow.

Test Plan

load the page, the arrows should be gone per the screenshots

Diff Detail

Repository

rCOMM Comm

Branch

inline-side-bar-ENG-530

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• benschac created this revision.Apr 29 2022, 11:09 AM

Herald added subscribers: • abosh, • karol, • adrian, ashoat. · View Herald TranscriptApr 29 2022, 11:09 AM

remove variable

• benschac retitled this revision from [web] [remove] [ENG-530] remove arrow from inline reply. to [web] [remove] [ENG-530] remove arrow from inline reply.Apr 29 2022, 11:11 AM

• benschac edited the summary of this revision. (Show Details)

Harbormaster completed remote builds in B8701: Diff 12120.Apr 29 2022, 11:13 AM

Harbormaster completed remote builds in B8702: Diff 12121.Apr 29 2022, 11:17 AM

• benschac requested review of this revision.Apr 29 2022, 11:17 AM

• benschac added a parent revision: D3879: [lib,native,web] move inline sidebar text to shared hook in lib.May 2 2022, 8:16 AM

Going to close this diff in favor of: D3839

• benschac abandoned this revision.May 2 2022, 2:46 PM