OpenCloud
/
riot-web
mirror of https://github.com/vector-im/riot-web
1
0
Fork 0
Code Issues Releases Wiki Activity

Don't include empty badge container in minimized view 

Fixes https://github.com/vector-im/riot-web/issues/14294

It takes up space, and it won't hold anything anyways.
pull/21833/head
Travis Ralston 2020-07-02 14:11:31 -06:00
parent 9fec5c98af
commit 0d9ce0721f
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/components/views/rooms/RoomTile2.tsx
View File

@ -369,11 +369,15 @@ export default class RoomTile2 extends React.Component<IProps, IState> {
let badge: React.ReactNode; let badge: React.ReactNode;
if (!this.props.isMinimized) { if (!this.props.isMinimized) {
badge = <NotificationBadge badge = (
notification={this.state.notificationState} <div className="mx_RoomTile2_badgeContainer">
forceCount={false} <NotificationBadge
roomId={this.props.room.roomId} notification={this.state.notificationState}
/>; forceCount={false}
roomId={this.props.room.roomId}
/>
</div>
);
} }
// TODO: the original RoomTile uses state for the room name. Do we need to? // TODO: the original RoomTile uses state for the room name. Do we need to?
@ -429,9 +433,7 @@ export default class RoomTile2 extends React.Component<IProps, IState> {
> >
{roomAvatar} {roomAvatar}
{nameContainer} {nameContainer}
<div className="mx_RoomTile2_badgeContainer"> {badge}
{badge}
</div>
{this.renderNotificationsMenu()} {this.renderNotificationsMenu()}
{this.renderGeneralMenu()} {this.renderGeneralMenu()}
</AccessibleButton> </AccessibleButton>