Skip to content

Conversation

@jeda
Copy link
Member

@jeda jeda commented Oct 5, 2024

This Pull Request contains multiple changes that I did locally over the years, either as tests or quick fixes to get instances working for myself or friends.

They were not committed back then due to missing polish or merely lack of testing. But given the slightly broken nature of Cryb 1, and Cryb 2 still being a WIP given my limited time/bandwidth for the past years, I figured I'd take a focus on getting this done and improve the stability and quality of what we've currently got.

Included here:

  • Overhauled the player:
    • Multiple changes to make the player more reliable
    • Reduce latency by negotiating zero playout delay
    • Use a video element instead of canvas for Picture in Picture
    • Fix audio on Chromium-based browsers by asking for stereo
    • Fixes some mouse control bugs
    • Grammar fixes
  • jsmpeg removed entirely (@cryb/aperture is deprecated, and was a terrible idea since the beginning)
  • Updated dependencies, including latest Nuxt.js v2 (which is now deprecated) + @nuxt/bridge
  • Switched from yarn to pnpm
  • Other QoL fixes/changes

On this commit:
- Migrated to latest Nuxt.js v2 and use @nuxt/bridge
- Include Janus as a dependency instead of importing
- Remove remains of JSMpeg/aperture
- Switch to pnpm instead of yarn
Also includes a few fixes from the past commit, and a few bits I missed.
On this commit:
- Multiple changes to make the player more reliable
- Reduce latency by negotiating zero playout delay
- Use a video element instead of canvas for PiP (why did I use canvas back then?)
- Fix audio on Chromium-based browsers by asking for stereo
- Fixes some control/mouse bugs
- Grammar
- Also remove some unused files with the @nuxt/bridge migration
- And fix eslint

This isn't the best code yet, but I am just pushing old fixes/changes from years ago.
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