Paths

Table of Contentst

[web] Introduce close button in `Label` component
ClosedPublic
Actions

Authored by • jacek on Apr 29 2022, 4:46 AM.

Details

Reviewers

tomek
• benschac
atul

Commits

rCOMM22df63a928fd: [web] Introduce close button in `Label` component

Summary

Introduce close action for Label component. The aim is to use in MembersModal in the list of pending users to add to the thread.

Without close action:

Screenshot_Google Chrome_2022-04-29_135216.png (47×113 px, 4 KB)

With:

Test Plan

Currently used Labels don't change because introduced property is optional and will be used in the next diff.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• jacek created this revision.Apr 29 2022, 4:46 AM

Herald added subscribers: • abosh, • karol, atul and 2 others. · View Herald TranscriptApr 29 2022, 4:46 AM

Harbormaster completed remote builds in B8693: Diff 12112.Apr 29 2022, 4:50 AM

• jacek requested review of this revision.Apr 29 2022, 4:50 AM

• jacek edited the summary of this revision. (Show Details)Apr 29 2022, 4:52 AM

• jacek added reviewers: • benschac, atul.

• jacek added a child revision: D3876: [web] Display list of pending users to add in `AddMembersModal`.Apr 29 2022, 6:11 AM

tomek accepted this revision.Apr 29 2022, 7:56 AM

This revision is now accepted and ready to land.Apr 29 2022, 7:56 AM

rebase

Harbormaster completed remote builds in B8839: Diff 12323.May 6 2022, 1:48 AM

Closed by commit rCOMM22df63a928fd: [web] Introduce close button in `Label` component (authored by • jacek). · Explain WhyMay 6 2022, 1:55 AM

This revision was automatically updated to reflect the committed changes.

• jacek added a commit: rCOMM22df63a928fd: [web] Introduce close button in `Label` component.

• karol removed a subscriber: • karol.May 6 2022, 3:54 AM

Revision Contents
Changeset List

Path

Size

web/

components/

label.css

10 lines

label.react.js

15 lines

Diff 12334

View Options

web/components/label.css