-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
Set blurred background from external api #15131
base: master
Are you sure you want to change the base?
Set blurred background from external api #15131
Conversation
Hi, thanks for your contribution! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the UI there are two types of blur effects, maybe the function needs to be parametrized to be able to achieve that, wdyt?
I don't need it at the moment but it could be an interesting feature. I'll add it when moving the code into virtual-background |
a1803c3
to
9aa4e65
Compare
Add setBlurredBackground command to external api.
9aa4e65
to
8c8573c
Compare
Parameter to choose blur type added to the method |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe I didn't express my self correctly the previous time, we try to remove code out of conference.js and to avoid adding stuff there.
const tracks = APP.store.getState()['features/base/tracks']; | ||
const videoTrack = getLocalVideoTrack(tracks)?.jitsiTrack; | ||
const muted = this.isLocalVideoMuted(); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can move this in toggleBlurredBackgroundEffect and dispatch it directly in API.js.
Add setBlurredBackground command to external api