Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Make the "Add people" button in the room-list act like an invite button while in a space #6274

Conversation

SimonBrandner
Copy link
Contributor

@SimonBrandner SimonBrandner commented Jun 27, 2021

Fixes element-hq/element-web#17792

Peek.2021-06-28.19-25.mp4

Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
@t3chguy
Copy link
Member

t3chguy commented Jun 27, 2021

What does it do if you don't have invite perms?

@SimonBrandner
Copy link
Contributor Author

What does it do if you don't have invite perms?

Breaks, I assume...

@SimonBrandner SimonBrandner marked this pull request as draft June 27, 2021 14:41
Signed-off-by: Šimon Brandner <[email protected]>
@SimonBrandner SimonBrandner marked this pull request as ready for review June 27, 2021 15:14
@t3chguy t3chguy requested review from a team June 28, 2021 07:25
Copy link
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code wise looks good. Couple of review points raised for design's benefit.

@@ -117,9 +118,24 @@ const TAG_AESTHETICS: ITagAestheticsMap = {
sectionLabel: _td("People"),
isInvite: false,
defaultHidden: false,
addRoomLabel: _td("Start chat"),
addRoomLabel: _td("Add people"),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Flagging this for design, should this stay as Start chat for when the Home space is being viewed?

src/components/views/rooms/RoomList.tsx Outdated Show resolved Hide resolved
Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
@SimonBrandner SimonBrandner added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Aug 4, 2021
@t3chguy
Copy link
Member

t3chguy commented Oct 29, 2021

This will be superseded by changes for element-hq/element-web#19526

@t3chguy t3chguy closed this Oct 29, 2021
@SimonBrandner SimonBrandner deleted the feature/start-chat-spaces/17792 branch October 29, 2021 11:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The start chat button should be more utilized in spaces
2 participants