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

Zoom feature #61

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

MuhammadjonTohirov
Copy link
Contributor

Zoom feature, that allows user to select portion of the screen and crops frames while streaming buffers.
It works only in RTCCustomFrameCapturer.

@MuhammadjonTohirov
Copy link
Contributor Author

MuhammadjonTohirov commented Feb 8, 2025

demo.mp4

This is from web server side. if developer wants to add zoom out or in features they can. because one can give a frame to videoCapturer so it will crop that portion and sends to the server and it looks like zooming something.

@mekya
Copy link
Contributor

mekya commented Feb 21, 2025

hi @MuhammadjonTohirov ,

Thank you for the implementation. I have had a review for this PR. We can use this functionality in the SDK for sure.

We want to control the zoom of the camera. Your current implementation makes sense and we can use for screensharing.

  • I'm asking you
    • To provide simple zoom in and zoom out functions for the camera into the SDK.
    • Please send another PR directly forked current master so that we can merge this PR and new PR separately.

If you need further clarity, let me know

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants