Skip to content
This repository has been archived by the owner on Sep 30, 2020. It is now read-only.

Sync Volume from Room Player to Preview Player #295

Open
wants to merge 3 commits into
base: staging
Choose a base branch
from

Conversation

mtomcal
Copy link

@mtomcal mtomcal commented Jul 3, 2016

Hi all, this is my first PR for this project. Please let me know what you think and how things go around here. Thanks!

  • Added or augmented setVolume on both Youtube and Soundcloud players.
  • In Youtube player, needed to add this.volume to save any volume settings before player is initialized. Once initialized and "ready" it sets Youtube player to this.volume.
  • Soundcloud player was more forgiving on setting volume for a sound is initialized on SoundManager
  • Added setting slider value to the volume level

Based off Issue #286 for fixing preview volume behavior enhancement.

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

Successfully merging this pull request may close these issues.

1 participant