Skip to content

fix(backgound): Move virtual background option from conversation to a…#17329

Merged
nickvergessen merged 1 commit intomainfrom
bugfix/17284/migrate-background-to-app-level
Mar 11, 2026
Merged

fix(backgound): Move virtual background option from conversation to a…#17329
nickvergessen merged 1 commit intomainfrom
bugfix/17284/migrate-background-to-app-level

Conversation

@nickvergessen
Copy link
Copy Markdown
Member

@nickvergessen nickvergessen commented Mar 9, 2026

…pp level

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

…pp level

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen added this to the 🏖️ Next Major (34) milestone Mar 9, 2026
@nickvergessen nickvergessen self-assigned this Mar 9, 2026
@nickvergessen nickvergessen added bug papercut feature: settings ⚙️ Settings and config related issues feature: frontend 🖌️ "Web UI" client feature: call 📹 Voice and video calls labels Mar 9, 2026
Comment thread src/init.js
'videoInputDevicePreferred', // Not needed anymore

// Talk 33
'virtualBackgroundBlurStrength_', // Migration from conversation level to Talk level settings
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We're only using blurValue: VIRTUAL_BACKGROUND.BLUR_STRENGTH.DEFAULT across the codebase, maybe we simply drop virtualBackgroundBlurStrength (without underscore) completely and only rely on a constant?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can do that in a follow up, just in case backlash is bigger then assumed

Copy link
Copy Markdown
Contributor

@DorraJaouad DorraJaouad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Antreesy btw older migrations are old enough IMO, we can clear them out soon?

@nickvergessen
Copy link
Copy Markdown
Member Author

btw older migrations are old enough IMO, we can clear them out soon?

I would keep them, in case a user updates over multiple versions. Doesn't really hurt

@nickvergessen nickvergessen merged commit 3c74c29 into main Mar 11, 2026
55 checks passed
@nickvergessen nickvergessen deleted the bugfix/17284/migrate-background-to-app-level branch March 11, 2026 10:26
@nickvergessen
Copy link
Copy Markdown
Member Author

/backport to stable33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug feature: call 📹 Voice and video calls feature: frontend 🖌️ "Web UI" client feature: settings ⚙️ Settings and config related issues papercut

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Set default background image for new calls

3 participants