-
Notifications
You must be signed in to change notification settings - Fork 739
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
Fully support Android 11 Conversations #1722
Comments
re: the second item - it looks like room and person icons aren't being cropped before being shown in the notification. Android crops them for us, but it means (it looks like) priority notification icons are being shown in the status bar without any crop at all. (Android 11 is still in beta so I don't know if the icons will get automatically cropped in the future) |
As of Beta 3 icons are cropped into a circle automatically (except the behaviour is weird with non-square icons -- probably just a temporary Android bug). Looks like 2 can be disregarded then. (edit -- interestingly enough, icons are cropped into a circle only with full conversation features, i.e. on my fork but not |
Really looking forward to this one. Do we have an ETA? |
Reading around it seems like the first task has been done (priority conversations, ..) in #1809 and/or #3313, but it does not seem supported at the moment (checked on two phones, latest release, latest debug) EDIT : Looked a bit more into this - it looks like it is actually an OS regression on Samsung's flavor of Android. It seems to work fine on a Google Pixel device. |
It seems like there are a few things that need to be done to fully support Android 11's new Conversations interface:
Not to mention it looks a lot better when they're not grouped. Here are some examples of what Element/RiotX notifications actually look like on Android 11:
(the last two are from an older version, but the notification style hasn't changed at all since then)
It might make sense to carry this down to older Android versions too. I don't see why notifications from separate rooms need to be grouped together; many other messaging platforms don't do this.
The text was updated successfully, but these errors were encountered: