Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Harmonise Room Notification Settings with Android/Web #4459

Closed
langleyd opened this issue Jun 23, 2021 · 0 comments · Fixed by #4458
Closed

Harmonise Room Notification Settings with Android/Web #4459

langleyd opened this issue Jun 23, 2021 · 0 comments · Fixed by #4458
Assignees

Comments

@langleyd
Copy link
Member

langleyd commented Jun 23, 2021

This issue is to track improvements to room level notification settings that bring more consistency across platforms.

The entry point for changing room level notification settings will move up one level from room settings to room info:
image

Rather than just a toggle to 'Mute Notifications' users can now choose from:
image

Note on a change in behaviour: In the old behaviour where we just had an on/off mute toggle, the off/muted state actually set the state to "Keywords and Mentions". Under these new options None does actually map to the None state( in line with the other clients.)

As keyword notifications are not supported on encrypted rooms for iOS when changing notifications they only options for available will be:
image

with an accompanying message:
image

When room notifications settings are changed via the slide interaction on a room list:
image

The room settings view will be presented modally and include a user avatar for context:
image

The following message/link will be added on the room notifications settings screen once we have made the improvements to the account level settings:
image

@langleyd langleyd self-assigned this Jun 23, 2021
@langleyd langleyd changed the title Harmonize Notification Settings with Android/Web Harmonise Room Notification Settings with Android/Web Jun 24, 2021
@langleyd langleyd linked a pull request Aug 27, 2021 that will close this issue
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant