remove arrows from native chat screen, per the design. Follow up diff will implement design
https://www.figma.com/file/L675ETKDnGaSwlpZAw4MIC/Mobile-App?node-id=344%3A491
https://linear.app/comm/issue/ENG-530/inline-sidebar-re-style
Differential D3897 Authored by • benschac on May 3 2022, 7:27 AM.
Details Summary remove arrows from native chat screen, per the design. Follow up diff will implement design https://www.figma.com/file/L675ETKDnGaSwlpZAw4MIC/Mobile-App?node-id=344%3A491 Test Plan Open simulator, go to a chat with sidebars, the arrows should be gone.
Diff Detail
Event TimelineHerald added subscribers: • abosh, • karol, • adrian. · View Herald TranscriptMay 3 2022, 7:27 AM2022-05-03 07:27:07 (UTC-7) • benschac edited the summary of this revision. (Show Details)May 3 2022, 7:28 AM2022-05-03 07:28:09 (UTC-7) • benschac added a parent revision: D3894: [lib] [refactor] [ENG-530] remove list of users string in reply. Harbormaster completed remote builds in B8736: Diff 12170.May 3 2022, 7:32 AM2022-05-03 07:32:27 (UTC-7) This revision is now accepted and ready to land.May 3 2022, 9:36 AM2022-05-03 09:36:08 (UTC-7) Harbormaster completed remote builds in B8823: Diff 12292.May 5 2022, 9:18 AM2022-05-05 09:18:06 (UTC-7)
Revision Contents
Diff 12292 native/chat/inline-sidebar.react.js
|
Do we really want to make this change in production? Should we consider gating it on isStaffRelease?