Make notification settings copy/layout more user-friendly (#34120)
* Make notification settings copy/layout more user-friendly and closer to EX. * Fix E2E tests.
This commit is contained in:
@@ -532,7 +532,7 @@ test.describe("Room list", () => {
|
|||||||
|
|
||||||
await app.viewRoomById(roomId);
|
await app.viewRoomById(roomId);
|
||||||
await app.settings.openRoomSettings("Notifications");
|
await app.settings.openRoomSettings("Notifications");
|
||||||
await page.getByText("Off").click();
|
await page.getByText("Mute").click();
|
||||||
await app.settings.closeDialog();
|
await app.settings.closeDialog();
|
||||||
|
|
||||||
const room = roomListView.getByRole("option", { name: "silent" });
|
const room = roomListView.getByRole("option", { name: "silent" });
|
||||||
|
|||||||
BIN
Binary file not shown.
|
Before Width: | Height: | Size: 114 KiB After Width: | Height: | Size: 114 KiB |
@@ -89,9 +89,9 @@ const RULE_DISPLAY_ORDER: string[] = [
|
|||||||
RuleId.EncryptedMessage,
|
RuleId.EncryptedMessage,
|
||||||
|
|
||||||
// Mentions
|
// Mentions
|
||||||
RuleId.ContainsDisplayName,
|
|
||||||
RuleId.ContainsUserName,
|
RuleId.ContainsUserName,
|
||||||
RuleId.AtRoomNotification,
|
RuleId.AtRoomNotification,
|
||||||
|
RuleId.ContainsDisplayName,
|
||||||
|
|
||||||
// Other
|
// Other
|
||||||
RuleId.InviteToSelf,
|
RuleId.InviteToSelf,
|
||||||
|
|||||||
@@ -231,7 +231,7 @@ export default class NotificationsSettingsTab extends React.Component<IProps, IS
|
|||||||
value: RoomNotifState.Mute,
|
value: RoomNotifState.Mute,
|
||||||
label: (
|
label: (
|
||||||
<>
|
<>
|
||||||
{_t("common|off")}
|
{_t("common|mute")}
|
||||||
<div className="mx_NotificationSettingsTab_microCopy">
|
<div className="mx_NotificationSettingsTab_microCopy">
|
||||||
{_t("notifications|mute_description")}
|
{_t("notifications|mute_description")}
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
Reference in New Issue
Block a user