Adding bubbles support #5554

arlexTech · 2025-11-14T12:20:13Z

🖼️ Screenshots

General settings	Conversation Info

VID_20251114124529.mp4

Record_2025-11-14-12-44-40.mp4

🚧 TODO

Bubble Works
Activating bubbles in settings
Activating or not bubbles for each conversation
Use Conversation Avatar for bubble icon
Multiple bubbles for multiple conversations
Embedded view
Create a bubble notification upon received message

🏁 Checklist

⛑️ Tests (unit and/or integration) are included or not needed
🔖 Capability is checked or not needed
🔙 Backport requests are created or not needed: /backport to stable-xx.x
📅 Milestone is set
🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt

mahibi · 2025-11-17T08:54:43Z

thank you @arlexTech
we will review soon!

mahibi

Thank you @arlexTech ! 👍
The bubbles itself work for me, however there are some things that must be changed:

Don't show conversation list and modify back button handling

I strongly suggest to not open the conversation list when pressing back button in bubbled mode.
So the bubbled view should not show the conversation list at all but only single conversations. Otherwise i'm concerned of bugs when having multiple instance of the ConversationListActivity.

For the back button in bubbled mode, either:

just close the bubble (This is how Telegram does it)
or replace the back button with a the talk icon which will open the app in normal mode and the conversation list is shown (This is how signal does it and it'S also my preferred solution)

Avoid multiple instance of ChatActivity for same conversation

Also, we should make sure that there are not multiple instances of ChatActivity for the same conversation.

Right now, the PR has a bug that multiple instances are shown when "open in app" is used.
https://developer.android.com/develop/ui/views/notifications/bubbles#launching-activities

But also we should not allow to have 2 instances for the same conversation when one is opened in normal mode and one in bubble mode.
This is very important to avoid bugs in the chat repository (OfflineFirstChatRepository)!

Signal solved it this way: Whenever there is a bubble for a chat and the chat is also opened in the normal mode, the bubble closes.
We should solve it the same way and try to use only one instance of ChatActivity for a conversation.

app/src/main/java/com/nextcloud/talk/chat/BubbleActivity.kt

app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt

app/src/main/java/com/nextcloud/talk/jobs/NotificationWorker.kt

mahibi · 2025-11-20T15:13:06Z

app/src/main/java/com/nextcloud/talk/jobs/NotificationWorker.kt

+        // Use a consistent ID based on the room token to avoid duplicate bubbles
        val systemNotificationId: Int =
-            activeStatusBarNotification?.id ?: calculateCRC32(System.currentTimeMillis().toString()).toInt()
+            activeStatusBarNotification?.id ?: NotificationUtils.calculateCRC32(pushMessage.id!!).toInt()


i am not 100% if using the room token could cause unwanted side effects. will have to think about and test when i have better internet connection ;)

mahibi · 2025-11-20T15:29:32Z

@rapterjet2004 @sowjanyakch please also do a deep review. E.g. specially check if former notification handling is affected by the changes.

app/src/main/java/com/nextcloud/talk/jobs/NotificationWorker.kt

arlexTech · 2025-11-22T19:29:59Z

I'll work on it this week (don't mind the review request i miss clicked)

mahibi · 2025-11-24T08:30:17Z

I'll work on it this week (don't mind the review request i miss clicked)

Sure, thanks. Let us know if there are any questions or help is needed!

github-actions · 2025-11-29T02:21:57Z

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

…when enabled

arlexTech · 2025-12-06T16:35:48Z

For the back button in bubbled mode I replaced it by the talk icon in bubble mode which open the app in normal mode on the conversation list

as onBackPressed is deprecated we want to avoid it and use onBackPressedDispatcher

fixed

multiple instances of the app are opened when "open is app" is used. https://developer.android.com/develop/ui/views/notifications/bubbles#launching-activities

should be fixed, I couldn't open multiple anymore

please try to extract the above code into a method

done

why is this removed?

because it was preventing bubbles to show up but it should be fixed now

I wanted the handleOnBackPressed function to minimize the bubble but I tried for hours and never managed to make it work correctly, if anyone has an idea of how to do it, I'm open to try anything at this point

Tell me if i can do anything more to merge it

arlexTech · 2025-12-06T16:35:59Z

But also we should not allow to have 2 instances for the same conversation when one is opened in normal mode and one in bubble mode.
This is very important to avoid bugs in the chat repository (OfflineFirstChatRepository)!
Signal solved it this way: Whenever there is a bubble for a chat and the chat is also opened in the normal mode, the bubble closes.
We should solve it the same way and try to use only one instance of ChatActivity for a conversation.

I didn't see this part, I wonder how Messenger deal with that because it can have the bubble and the chat open at the same time

mahibi

Thanks to implement the suggestions @arlexTech 👍 We appreciate your efforts!

I commented on some things that must be changed.

There were changes on master in NotificationWorker that conflict with your branch. Could you resolve it?

Please also see the codacy warnings at
https://app.codacy.com/gh/nextcloud/talk-android/pull-requests/5554/issues

and sign your commits like described in
https://github.com/nextcloud/talk-android/pull/5554/checks?check_run_id=57331453438

If you need any help or nextcloud employees should take over, just let us know.

mahibi · 2025-12-15T16:26:12Z

app/src/main/java/com/nextcloud/talk/jobs/NotificationWorker.kt

+        // Use a consistent ID based on the room token to avoid duplicate bubbles
        val systemNotificationId: Int =
-            activeStatusBarNotification?.id ?: calculateCRC32(System.currentTimeMillis().toString()).toInt()
+            activeStatusBarNotification?.id ?: NotificationUtils.calculateCRC32("${signatureVerification.user!!.id}:${pushMessage.id!!}").toInt()


The content of the notification history is not cleared when the conversation was opened:

Receive a notification "a"

Open it via click on notification or open the app and go to conversation

Receive another notification "b"

Bug:
"a" and "b" is shown

Expected:
only "b" is shown

This will be caused by the systemNotificationId which does not change anymore because it's the roomToken. This should be unique (like a timestamp that it was before).
For shortcutId it should make sense to have the roomToken.

mahibi · 2025-12-15T16:28:40Z

app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt

            }

+            R.id.create_conversation_bubble -> {
+                createConversationBubble()


When bubble exists and and in normal chat mode "Create bubble" for the same conversation is chosen, then there are 2 bubbles for the same conversation.

sowjanyakch

Thank you for PR @arlexTech

The inbuilt emoji picker does not appear when tapping the emoji icon in the bubble notification chat view.

arlexTech · 2025-12-19T12:54:56Z

Thank you for your feedback, i try to fix all of this and if I'm hard stuck I'll ask for help

Adding bubbles support

726b0ab

AndyScherzinger requested review from mahibi, rapterjet2004 and sowjanyakch November 14, 2025 12:56

AndyScherzinger added enhancement New feature or request 3. to review Waiting for reviews labels Nov 14, 2025

github-advanced-security bot found potential problems Nov 14, 2025

View reviewed changes

app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt Fixed Show fixed Hide fixed

fix ktlintCheck detekt testGplayDebugUnitTest

49b4b93

github-advanced-security bot found potential problems Nov 14, 2025

View reviewed changes

app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt Show resolved Hide resolved

mahibi requested changes Nov 20, 2025

View reviewed changes

mahibi reviewed Nov 20, 2025

View reviewed changes

app/src/main/java/com/nextcloud/talk/jobs/NotificationWorker.kt Show resolved Hide resolved

arlexTech requested a review from mahibi November 22, 2025 19:27

github-actions bot added the feedback-requested label Nov 29, 2025

arlexTech added 5 commits December 6, 2025 15:50

remove deprecated onBackPressed and extracted code in function

eab6630

prevent second app instance and adding talk icon button in bubble mode

5240677

fix potential issue with room token

4dbec66

fix getLargeIcon preventing bubble notification

c72622e

bubbles off by default and lead to android bubble settings if needed …

b33937a

…when enabled

mahibi requested changes Dec 15, 2025

View reviewed changes

sowjanyakch requested changes Dec 16, 2025

View reviewed changes

Uh oh!

Adding bubbles support #5554

Are you sure you want to change the base?

Adding bubbles support #5554

Uh oh!

Conversation

arlexTech commented Nov 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🖼️ Screenshots

🚧 TODO

🏁 Checklist

Uh oh!

Uh oh!

Uh oh!

mahibi commented Nov 17, 2025

Uh oh!

mahibi left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Don't show conversation list and modify back button handling

Avoid multiple instance of ChatActivity for same conversation

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mahibi Nov 20, 2025

Choose a reason for hiding this comment

Uh oh!

mahibi commented Nov 20, 2025

Uh oh!

Uh oh!

arlexTech commented Nov 22, 2025

Uh oh!

mahibi commented Nov 24, 2025

Uh oh!

github-actions bot commented Nov 29, 2025

Uh oh!

arlexTech commented Dec 6, 2025 • edited by mahibi Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

arlexTech commented Dec 6, 2025

Uh oh!

mahibi left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mahibi Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

mahibi Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

sowjanyakch left a comment

Choose a reason for hiding this comment

Uh oh!

arlexTech commented Dec 19, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

arlexTech commented Nov 14, 2025 •

edited

Loading

mahibi left a comment •

edited

Loading

arlexTech commented Dec 6, 2025 •

edited by mahibi

Loading

mahibi left a comment •

edited

Loading