-
-
Notifications
You must be signed in to change notification settings - Fork 834
Conversation
There was a problem hiding this 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; |
There was a problem hiding this comment.
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
?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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
src/i18n/strings/en_EN.json
Outdated
@@ -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" |
There was a problem hiding this comment.
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
element-hq/element-web#7607
A bit hacky in places, sorry.