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

Add new android notification features (Chat bubbles, priority conversations) #2734

Closed
33Fraise33 opened this issue Jan 28, 2021 · 21 comments · Fixed by #3313
Closed

Add new android notification features (Chat bubbles, priority conversations) #2734

33Fraise33 opened this issue Jan 28, 2021 · 21 comments · Fixed by #3313
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements

Comments

@33Fraise33
Copy link

33Fraise33 commented Jan 28, 2021

Is your feature request related to a problem? Please describe.
When opening the android notification settings it gives a headsup about missing features:

"Element (Riot.im) doesn't support most conversation features. You can't set a conversation as priority, and conversations won't appear as floating bubbles"

This is a native android feature described working here: https://www.androidcentral.com/how-use-chat-bubbles-android-11

Describe the solution you'd like
I'd like this native feature to be implemented so chat bubbles can be used. This is not to be confused with #2388 , #1805 and #979. They are talking about the chat view in app, this is a bubble overlay showing over all apps like Facebook messenger did first.

Describe alternatives you've considered
None existant. This makes quickly looking something up and copying it much easier. Also quickly jumping to a chat while doing something else.

Additional context
/

@33Fraise33 33Fraise33 added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Jan 28, 2021
@AquaWolf
Copy link

I also want this feature. I looked a bit in the code perhaps I can program this but still need some more orientation in the code.

@AquaWolf
Copy link

Added some lines to enable shortcuts and therefore priority conversation but not sure if I missed something:
dd7c264

@AquaWolf AquaWolf mentioned this issue May 11, 2021
6 tasks
@AquaWolf
Copy link

I cannot get these bubbles working. As start i added conversation priority in this PR so the notifications can appear in top of other notifications.

@hanthor
Copy link
Contributor

hanthor commented Jun 16, 2021

I wonder why the bubbles don't work. I have noticed not many Android apps implement the bubbles feature.

@Peterwmoss
Copy link

Any news on this? I would really like to see chat bubbles :)

@ShadowJonathan
Copy link
Contributor

Element-web recently got chat bubbles, so i expect Design to pick up on it and give it to Mobile apps soon

@kevincox
Copy link

kevincox commented Aug 9, 2021

@ShadowJonathan I think you are confusing two different issues. This is talking about Android Chat Bubbles which is about using the system chat-app overlay feature so that Element is always on-screen. The feature recently added to Element Web is about changing the look of the messages inside of Element.

@ShadowJonathan
Copy link
Contributor

Oh whoops, yeah, i dun goofed for a second 😅

@AquaWolf
Copy link

AquaWolf commented Sep 8, 2021

Sorry for this long time not working on this issue actually, I will give this a next try because I really want this feature. I will clean up my PR and try to get these chat bubbles working. I cannot give any timeline for this. Does anybody know who to ping to review my PR after my clean up? I tried to contact @bmarty in matrix.org but I didn't get any response from him :(

@AquaWolf
Copy link

AquaWolf commented Sep 8, 2021

Because of native Android 11 chat bubbles I need to implement some kind of view for the chat window in the system overlay any advise for a ui for that?

@AquaWolf
Copy link

Thank you @bmarty for reviewing and merging happy to see this feature in the App 👍

@Peterwmoss
Copy link

Peterwmoss commented Oct 15, 2021

Nice to see updates on this, but does this mean that bubbles are still missing?

@AquaWolf
Copy link

Yes Bubbles still missing my PR just included (priority) conversation feature

@Peterwmoss
Copy link

Issue should then be reopened, since it doesn't cover the entirety of the issue? Or should there be a separate issue for chat bubbles?

@SimonBrandner
Copy link

Issue should then be reopened, since it doesn't cover the entirety of the issue? Or should there be a separate issue for chat bubbles?

That is a different issue - #979

@Peterwmoss
Copy link

No, this is not internal to the app. This is an android feature - just as @kevincox mentioned here

@hanthor
Copy link
Contributor

hanthor commented Oct 15, 2021 via email

@SimonBrandner
Copy link

Ah, you're right I got confused and thought the PR added the Android chat bubbles and I thought you were asking about the bubble-themed chat, sorry! 😅 In that case, yes, this should be re-opened/new issue should be opened. I prefer the latter, personally

@AquaWolf
Copy link

Yep you're right there must be implemented some special Activity UI for Android chat bubbles. Would be nice to reopen or split this issue just to keep in mind that the chat bubbles are requested.

@abmantis
Copy link

@AquaWolf there's already an issue for chat bubbles: #1124 and also #1722

@th0mcat
Copy link
Contributor

th0mcat commented Oct 23, 2021

For reference, here is the commit to get it working on Signal Android:

signalapp/Signal-Android@e1bf232

There have been changes and fixes in other commits since then I believe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants