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

Redesign: room directory makeover #2519

Merged
merged 5 commits into from
Jan 29, 2019

Conversation

bwindels
Copy link
Contributor

element-hq/element-web#7607

A bit hacky in places, sorry.

roomdirectorymakeover

@bwindels bwindels changed the title Bwindels/roomdirectory makeover Redesign: room directory makeover Jan 29, 2019
@bwindels bwindels requested a review from a team January 29, 2019 16:26
@turt2live turt2live self-assigned this Jan 29, 2019
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

otherwise lgtm

}

.mx_RoomDirectory_searchbox {
display: table-cell;
vertical-align: middle;
flex: 1 !important;
Copy link
Member

Choose a reason for hiding this comment

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

do more specific styles help at all in this case? Something like .mx_RoomDirectory_dialogWrapper .mx_RoomDirectory_searchBox ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tried that, but the rule it's trying to override is very very specific (see _dharma.scss:260). The whole mx_textinput/ global input styles need to be better scoped anyways, so rather postpone this till then.

Copy link
Member

Choose a reason for hiding this comment

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

:(

+1 to postponing. It's not a major problem, just a thing we should avoid if possible

@@ -1497,5 +1497,6 @@
"If you didn't remove the recovery method, an attacker may be trying to access your account. Change your account password and set a new recovery method immediately in Settings.": "If you didn't remove the recovery method, an attacker may be trying to access your account. Change your account password and set a new recovery method immediately in Settings.",
"Failed to set direct chat tag": "Failed to set direct chat tag",
"Failed to remove tag %(tagName)s from room": "Failed to remove tag %(tagName)s from room",
"Failed to add tag %(tagName)s to room": "Failed to add tag %(tagName)s to room"
"Failed to add tag %(tagName)s to room": "Failed to add tag %(tagName)s to room",
"Join": "Join"
Copy link
Member

Choose a reason for hiding this comment

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

npm run i18n please to maintain ordering

@turt2live turt2live assigned bwindels and unassigned turt2live Jan 29, 2019
@bwindels bwindels merged commit 0cce912 into experimental Jan 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants