Skip to content

Commit 3085f56

Browse files
authored
Merge pull request #1359 from jjoseba/oppia-1647-multiple-audios
OPPIA-1647: Update JSInterface to take into account multiple audios
2 parents 0c0cf37 + 57870fd commit 3085f56

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

app/src/main/assets/js_injects/observe_audio_player.js

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,13 @@
11
(function(){
2-
var playButton = document.getElementById("play-icon");
3-
playButton.addEventListener("click", function() {
4-
var audioElement = document.querySelector("audio");
5-
var audioSource = audioElement.getAttribute("src");
6-
{{INTERFACE_EXPOSED_NAME}}.onPlayButtonClick(audioSource);
7-
audioElement.addEventListener("ended", function() {
2+
$(".audio-player-container, #audio-player-container").each(function(){
3+
container = $(this);
4+
var audioElem = container.find("audio")[0];
5+
container.find(".play-icon,#play-icon").on('click', function(){
6+
var audioSource = audioElem.getAttribute("src");
7+
{{INTERFACE_EXPOSED_NAME}}.onPlayButtonClick(audioSource);
8+
});
9+
audioElem.addEventListener("ended", function() {
10+
var audioSource = audioElem.getAttribute("src");
811
{{INTERFACE_EXPOSED_NAME}}.onAudioCompleted(audioSource);
912
});
1013
});

0 commit comments

Comments
 (0)