Thanks @akx we can fix these docstring issues and merge this. Just FYI, _js is intentionally undocumented because it is intended for internal use at this point, and it's API may change, until we close #2479

All the demos for this PR have been deployed at https://huggingface.co/spaces/gradio-pr-deploys/pr-3709-all-demos

Thanks @akx we can fix these docstring issues and merge this. Just FYI, _js is intentionally undocumented because it is intended for internal use at this point, and it's API may change, until we close #2479

The parameter is being used in the wild, which was exactly why I was confused about the lack of docstring...

Would you like me to remove that change from the PR, or maybe add "experimental, may change" or something to the doc?

The parameter is being used in the wild, which was exactly why I was confused about the lack of docstring...

Makes sense, I'll add some documentation which explicitly states that this is an experimental parameter.

Btw @akx can you allow maintainers to push to this PR? I'll fix the remaining docstrings

Hi @akx ! This looks good to me. Can you re-open the PR with permission for outside commits so that @abidlabs can fix the remaining docstrings? Thank you.

Btw @akx can you allow maintainers to push to this PR? I'll fix the remaining docstrings

Done. Sorry for the delay

Thanks @akx! Fixing the docstrings now

I'm running into some issues trying to push changes to this branch. Started a new PR with these changes (#3740)