Getting audio players to play their respective streams out of their element

asasass 2017-12-12 08:39:30 UTC #985

no.............

Paul_Wilkins 2017-12-12 08:39:38 UTC #986

If there isn't anything else that needs to be in that function, then close the function.

asasass 2017-12-12 08:42:25 UTC #987

Shouldn't play audio be first?

      function manageAudio(player, opts) {
        if (opts.playing) {
          pauseAudio(player);
        } else {
          playAudio(opts.playing); 
        }

Paul_Wilkins 2017-12-12 08:43:09 UTC #988

When the audio is already playing, and you click on the button, you want it to play the audio again?

Are you nuts?

asasass 2017-12-12 08:43:42 UTC #989

I'm confused.

asasass 2017-12-12 08:44:06 UTC #990

Isn't it already closed?

  function manageAudio(player, opts) {
        if (opts.playing) {
          pauseAudio(player);
        } else {
          playAudio(opts.playing); 
        }

Paul_Wilkins 2017-12-12 08:44:59 UTC #991

With the manageAudio() function, if the button is currently playing then it pauses the audio.
Otherwise, (because the button isn't currently playing), it plays the audio.

That's how the manageAudio() function works. Surely you can tell how it works from the code that's in the function.

Paul_Wilkins 2017-12-12 08:45:13 UTC #992

No it's not.

asasass 2017-12-12 08:45:51 UTC #993

It gets another curly brace?

Paul_Wilkins 2017-12-12 08:46:40 UTC #994

What do you mean another?

asasass 2017-12-12 08:47:03 UTC #995

You said it's not closed, that's how you close it.
https://jsfiddle.net/6ts492f6/462/

I don't understand.

How do you close it??

  function manageAudio(player, opts) {
        if (opts.playing) {
          pauseAudio(player);
        } else {
          playAudio(opts.playing); 
        }
}

Paul_Wilkins 2017-12-12 08:50:23 UTC #996

You close a function by using a closing curly brace.

It looks like you added one for that manageAudio() function, which does the job nicely, although it 's always better to indent the curly brace so that it vertically lines up with the start of the line that has the matching opening brace.

asasass 2017-12-12 08:51:48 UTC #997

https://jsfiddle.net/6ts492f6/466/

      function manageAudio(player, opts) {
        if (opts.playing) {
          pauseAudio(player);
        } else {
          playAudio(opts.playing);
        });
    }
  }

Paul_Wilkins 2017-12-12 08:53:21 UTC #998

Do you realise that most (if not all) of what you are adding will eventually need to be removed before the code works?
Most of what you are adding is invalid syntax, and the code won't even attempt to start until it's valid syntax.

asasass 2017-12-12 08:55:41 UTC #999

https://jsfiddle.net/6ts492f6/466/

Paul_Wilkins 2017-12-12 08:56:28 UTC #1000

What code do you currently have?

asasass 2017-12-12 08:56:54 UTC #1001

https://jsfiddle.net/6ts492f6/466/

Paul_Wilkins 2017-12-12 08:57:37 UTC #1002

The console is complaining about the parenthesis that you added most recently, so remove that.

asasass 2017-12-12 08:57:37 UTC #1003

SyntaxError: expected expression, got ')'

    function manageAudio(player, opts) {
        if (opts.playing) {
          pauseAudio(player);
        } else {
          playAudio(opts.playing);
        });
    }
  }

asasass 2017-12-12 08:58:08 UTC #1004

What line number?

ThemesPodcastArticlesPremium