feat(call): start 1-1 call from anywhere by Antreesy · Pull Request #17205 · nextcloud/spreed

Antreesy · 2026-02-24T17:46:25Z

☑️ Resolves

Fix 📍 Call from anywhere - Integration of calls into the avatar menu #15416

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

Entry points 🚧 WIP	Integration 🚧 WIP

🚧 Tasks

Feature related:

Technical:

Lazy load
- interceptor/trigger is ~9 kB
- integration ~ 1.25 Mb
Reduce size (where seems possible)
- Light-weight versions of existing components?
- Do not use TopBar - keep name and timer only
Limit PR changes to relevant
- Split unrelated changes to pre-requisites
- Split design / general components changes to pre-requisites

🏁 Checklist

🛠️ API Checklist

🚧 Tasks

~~Abstract profile actions~~
set priority

🏁 Checklist

⛑️ Tests (unit and/or integration) are included or not possible
📘 API documentation in docs/ has been updated or is not required
🔖 Capability is added or not needed

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

nickvergessen · 2026-04-22T14:07:06Z

+		$directIconUrl = $this->urlGenerator->getAbsoluteURL($this->urlGenerator->imagePath('spreed', 'icon-phone-dark.svg'));
+		$directCallUrl = $this->urlGenerator->linkToRouteAbsolute('spreed.Page.index') . '?callUser=' . $user->getUID() . '#direct-call';
+		$directAction = $this->actionFactory->newLinkAction($directIconUrl, $directTalkAction, $directCallUrl, Application::APP_ID);
+		$directAction->setPriority(9);


Higher is more important (shown)
I'd go with 30 now, so it's the option that shows outside

Suggested change

$directAction->setPriority(9);

$directAction->setPriority(30);

At least above 20, so it comes as an outside action before the profile?

Oh, I understood it should be below 'Chat with user' 🙈
Fine by me, but I'd align with designers

Worth mentioning: Hover cards with a nice layout nextcloud/server#30868

Any chance it's possible to change the order depending on where this menu is used? Calling is definitely more useful as an outside action than viewing profile in the contacts menu but in Talk it feels weird for the order to be "Call, view profile, local time, email, chat". If it's just in this contacts menu then it can be outside, but in Talk it would be nice to group all the "contacting" options together.

If not possible we just do calling as the highest priority and try to do the nice hover cards in the future 🤷

nickvergessen · 2026-04-22T14:14:54Z

I think we should use a camera icon like on start-call?

Or we use Talk for the call and a speech bubble for chat

Then it would be confusing?

Talk icon > open small popup with a call

speech buble icon > open Talk app

Other ideas:

video_camera_front: https://fonts.google.com/icons?selected=Material+Symbols+Outlined:video_camera_front:FILL@0;wght@400;GRAD@0;opsz@24&icon.query=camera&icon.size=24&icon.color=%235f6368%5C

video_chat: https://fonts.google.com/icons?selected=Material+Symbols+Outlined:video_chat:FILL@0;wght@400;GRAD@0;opsz@24&icon.query=camera&icon.size=24&icon.color=%235f6368%5C

picture_in_picture: https://fonts.google.com/icons?selected=Material+Symbols+Outlined:picture_in_picture:FILL@0;wght@400;GRAD@0;opsz@24&icon.query=video&icon.size=24&icon.color=%235f6368%5C

I'd vote for a simple video cam icon. Second preference would be video-chat

DorraJaouad

I reviewed based on smoke tests I did manually to address mainstream issues. After this, I will go into more details layer (minor issues)

DorraJaouad · 2026-04-23T10:13:41Z

+	// ignore external links
+	if (url.origin !== location.origin) {
+		return
+	}


It will ignore when using different protocols too (https vs http), what about

const url = new URL(href, location.href) if (url.host !== location.host) { return }

DorraJaouad · 2026-04-23T10:19:21Z

+		store.dispatch('addConversation', conversation)
+		const token = conversation.token
+
+		if (!window.OCA.Talk) {


What about when you already have an instance:

Block any second call of the same user

Block any other call of another user

DorraJaouad · 2026-04-23T10:22:49Z

+	} catch (exception) {
+		joiningConversation.value = false
+
+		showError(t('spreed', 'Error occurred when joining the conversation'))
+
+		console.error(exception)
+
+		return
+	}


In case an exception happens, the overlay needs to be unmounted as there is no close button

DorraJaouad · 2026-04-23T10:24:01Z

+function openInNewTab() {
+	const url = generateAbsoluteUrl('/call/{token}#direct-call', { token: props.token })
+	window.open(url, '_blank')
+}


The overlay needs to be unmounted, currently it just freezes and stays there

DorraJaouad · 2026-04-23T10:34:10Z

+ */
+
+import type { Conversation } from './types/index.ts'
+


Suggested change

import './init.js'

Otherwise nothing is intercepted :)
Also can you please rename it to mainFloatingCall ? We used this prefix in other instances

Antreesy added this to the 🏖️ Next Major (34) milestone Feb 24, 2026

Antreesy self-assigned this Feb 24, 2026

Antreesy added enhancement feature: frontend 🖌️ "Web UI" client feature: talk-sidebar ⬅️ Sidebar integration of Talk into other apps like sharing and documents feature: call 📹 Voice and video calls labels Feb 24, 2026

Antreesy added this to 💬 Talk team Mar 13, 2026

Antreesy moved this to 🏗️ In progress in 💬 Talk team Mar 13, 2026

Antreesy force-pushed the feat/15416/call-from-anywhere branch 4 times, most recently from 3d933c2 to 9185c55 Compare April 15, 2026 12:02

nickvergessen reviewed Apr 16, 2026

View reviewed changes

Comment thread lib/Profile/TalkCallAction.php

nickvergessen reviewed Apr 16, 2026

View reviewed changes

Comment thread lib/Profile/TalkCallAction.php

Antreesy force-pushed the feat/15416/call-from-anywhere branch from 9185c55 to 3104c7f Compare April 21, 2026 17:35

nickvergessen reviewed Apr 22, 2026

View reviewed changes

Comment thread lib/ContactsMenu/Providers/CallProvider.php

Antreesy force-pushed the feat/15416/call-from-anywhere branch from 3104c7f to 906fffd Compare April 22, 2026 13:22

Antreesy added 6 commits April 22, 2026 15:52

fix(API): register 'direct-call' action

0a5a2e5

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

fix(router): better handle initial navigation in integrations

c7dd6fa

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

fix(signaling): cleanup signaling object after kill

5c1ddb7

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

wip: introduce floatingCallTrigger entrypoint

af78cfc

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

wip: introduce floatingCall initialization logic

bf75ee6

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

feat: introduce floating component

a2b7efb

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

Antreesy force-pushed the feat/15416/call-from-anywhere branch from 906fffd to a2b7efb Compare April 22, 2026 13:52

Antreesy marked this pull request as ready for review April 22, 2026 13:52

Antreesy requested a review from DorraJaouad April 22, 2026 13:52

Antreesy changed the title ~~feat(call): start 1-1 call from 'Talk to {person}' action~~ feat(call): start 1-1 call from anywhere Apr 22, 2026

nickvergessen reviewed Apr 22, 2026

View reviewed changes

DorraJaouad reviewed Apr 23, 2026

View reviewed changes

	$directAction->setPriority(9);
	$directAction->setPriority(30);

Conversation

Antreesy commented Feb 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🚧 Tasks

🏁 Checklist

🛠️ API Checklist

🚧 Tasks

🏁 Checklist

Uh oh!

Uh oh!

Uh oh!

Uh oh!

nickvergessen Apr 22, 2026 • 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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

DorraJaouad left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Antreesy commented Feb 24, 2026 •

edited

Loading

nickvergessen Apr 22, 2026 •

edited

Loading