starred/koel-koel

Fork 0

mirror of https://github.com/koel/koel.git synced 2026-04-26 01:06:00 +03:00

[GH-ISSUE #532] Music stops after first song on Firefox > v51 #376

New issue

Closed

opened 2026-02-26 02:33:00 +03:00 by kerem · 5 comments

kerem commented

2026-02-26 02:33:00 +03:00

Owner

Originally created by @X-Ryl669 on GitHub (Feb 5, 2017).
Original GitHub issue: https://github.com/koel/koel/issues/532

With latest firefox, when playing multiple songs in the queue, the first song plays fine but any other later song fails to play. This also happens with your demo.

It works with webkit latest version and firefox < 49.
There's no message in the javascript console, no exception whatsoever, the interface just looks like fine (the time is increasing etc...) but no sound is produced and the speaker icon is not overlaid on the browser's tab (as when audio is running).

Demo on player.js runs fine.

Originally created by @X-Ryl669 on GitHub (Feb 5, 2017). Original GitHub issue: https://github.com/koel/koel/issues/532 With latest firefox, when playing multiple songs in the queue, the first song plays fine but any other later song fails to play. This also happens with your demo. It works with webkit latest version and firefox < 49. There's no message in the javascript console, no exception whatsoever, the interface just looks like fine (the time is increasing etc...) but no sound is produced and the speaker icon is not overlaid on the browser's tab (as when audio is running). Demo on player.js runs fine.

kerem closed this issue

2026-02-26 02:33:00 +03:00

kerem commented

2026-02-26 02:33:01 +03:00

Author

Owner

@Diagamma commented on GitHub (Feb 6, 2017):

I have this issue too. By the way, it works perfectly with Firefox Developer Edition (v53.0a2).

@Diagamma commented on GitHub (Feb 6, 2017): I have this issue too. By the way, it works perfectly with Firefox Developer Edition (v53.0a2).

kerem commented

2026-02-26 02:33:01 +03:00

Author

Owner

@X-Ryl669 commented on GitHub (Feb 6, 2017):

Is there anything in the Firefox's changelog that could explain they corrected the bug ?
I've tried to report the bug on Mozilla, but it's almost impossible to find out if it's already there, or if it's corrected already, etc...

I just want to make sure it's fixed in the upcoming version 52 or 53 or if it's a side effect of the multi-process change they added (and in that case, figure out if the usage pattern of player.js in Koel is wrong)

@X-Ryl669 commented on GitHub (Feb 6, 2017): Is there anything in the Firefox's changelog that could explain they corrected the bug ? I've tried to report the bug on Mozilla, but it's almost impossible to find out if it's already there, or if it's corrected already, etc... I just want to make sure it's fixed in the upcoming version 52 or 53 or if it's a side effect of the multi-process change they added (and in that case, figure out if the usage pattern of player.js in Koel is wrong)

kerem commented

2026-02-26 02:33:01 +03:00

Author

Owner

@phanan commented on GitHub (Feb 8, 2017):

But Koel doesn't use player.js. I'd suspect something went wrong with document.querySelector('.plyr').addEventListener('ended' in playback.js, or the preload functionality. Will take a look when I have some time – sorry, I'm really busy recently.

@phanan commented on GitHub (Feb 8, 2017): But Koel doesn't use player.js. I'd suspect something went wrong with `document.querySelector('.plyr').addEventListener('ended'` in `playback.js`, or the preload functionality. Will take a look when I have some time – sorry, I'm really busy recently.

kerem commented

2026-02-26 02:33:01 +03:00

Author

Owner

@X-Ryl669 commented on GitHub (Feb 14, 2017):

Tested v52beta6 and it's working. I'll close this bug when official 52 is out and the issue does not happen.

@X-Ryl669 commented on GitHub (Feb 14, 2017): Tested v52beta6 and it's working. I'll close this bug when official 52 is out and the issue does not happen.

kerem commented

2026-02-26 02:33:01 +03:00

Author

Owner

@X-Ryl669 commented on GitHub (Mar 8, 2017):

Official version 52 is out, and it works. Thanks.

@X-Ryl669 commented on GitHub (Mar 8, 2017): Official version 52 is out, and it works. Thanks.

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/koel-koel#376

No description provided.

Rows
Columns