Skip to content

only use synchronized volume players#203

Merged
mherger merged 1 commit intoLMS-Community:public/7.9from
philippe44:syncVolume
Nov 5, 2018
Merged

only use synchronized volume players#203
mherger merged 1 commit intoLMS-Community:public/7.9from
philippe44:syncVolume

Conversation

@philippe44
Copy link
Contributor

Synchronize volume only amongst players which have the sync volume attribute set, including at stream startup (i.e. the sync master is not always used as the reference volume)

@philippe44
Copy link
Contributor Author

While discussing the HTTP keep-alive, I was reviewing my past submissions. Was this one wrong? Shall I remove it?

@mherger
Copy link
Contributor

mherger commented Nov 5, 2018

I'm sorry, never really looked into this one. What behaviour does it "fix"?

@mherger mherger merged commit 15f0461 into LMS-Community:public/7.9 Nov 5, 2018
@mherger
Copy link
Contributor

mherger commented Nov 5, 2018

Never mind... read up on the bug report and forum thread. Thanks!

@philippe44
Copy link
Contributor Author

Thanks - I hope I did not do anything silly. Obviously I have a poor track record in the quality of my submissions

@mherger
Copy link
Contributor

mherger commented Nov 5, 2018

No worries: the reviewer obviously didn't know better :-D

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