Differential D4122

[web] [feat] [ENG-536] style sidebars modal
AbandonedPublic
Actions

Authored by • benschac on May 24 2022, 5:29 PM.

Tags

None

Referenced Files

	F32099163: D4122.1765011746.diff
	Fri, Dec 5, 9:02 AM

Subscribers

Details

Reviewers

tomek
ashoat
atul

Summary

add basic styling to the sidebars modal

Test Plan

will add screen shots, there isn't design for this screen. I'll get feedback from @ashoat

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• benschac created this revision.May 24 2022, 5:29 PM

Herald added subscribers: • abosh, • adrian. · View Herald TranscriptMay 24 2022, 5:29 PM

• benschac added a parent revision: D4067: [web, lib, native] [refactor] [ENG-536] rename listData to sidebar.May 24 2022, 5:30 PM

Harbormaster completed remote builds in B9457: Diff 13095.May 24 2022, 6:13 PM

• benschac requested review of this revision.May 24 2022, 6:13 PM

Please attach screenshots before requesting review

web/chat/sidebar-modal-item.react.js
16	"Sidebar" is one word. @palys-swm gave you this feedback in D4066 and D4067, and it hasn't been addressed in either, so I'm a bit worried that you missed it. Requesting changes to make sure you grok this feedback and internalize it, so it doesn't have to be repeated again

This revision now requires changes to proceed.May 24 2022, 7:31 PM

@jacek I believe you handled this separately? It looks fine on prod.

atul abandoned this revision.Aug 15 2022, 5:37 PM

atul changed the author of this revision to • benschac.

atul edited reviewers, added: atul; removed: • benschac.

Revision Contents
Changeset List

Path

Size

web/

chat/

sidebar-modal-item.css

33 lines

sidebar-modal-item.react.js

54 lines

modals/

chat/

sidebar-list-modal.react.js

7 lines

1 line

Diff 13095

web/chat/sidebar-modal-item.css

Loading...

web/chat/sidebar-modal-item.react.js

Loading...

web/modals/chat/sidebar-list-modal.react.js

Loading...

web/theme.css

Loading...