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

Allow user to specify media options on request. #38

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

Conversation

allen-edia
Copy link

@allen-edia allen-edia commented Aug 16, 2017

Hi,

This is intended to address #30.

The application code can specify a 'get_media_options' function during construction, which is passed the default_media_options and returns an arbitrary options object.

In our case we could request the maximum possible resolution from Firefox using the following code, after which it is no longer capped at 640x.

            get_media_options: (default_media_options) => lodash.merge(
                {},
                default_media_options,
                {
                    // --- request maximum possible resolution:
                    // necessary for Firefox, at least.
                    video: {
                        width: 9999,
                        height: 9999,
                    }
                }
            )

User can provide an optional 'get_media_options' function to the options
object, which will take the default_media_options as a parameter and can
return an arbitrary object.

This is intended to allow, among other things, users requesting resolutions
other than the ones in the default_media_options.
@kirantpatil
Copy link

Yes, even in latest Firefox 58 on android, I am getting this issue, but not on chrome browser.

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