Skip to content

Commit 7cb169b

Browse files
authored
Disabled record and save buttons while music is playing (sugarlabs#4245)
* Disabled record and save buttons while music is playing * removed extra lines
1 parent cf84e3f commit 7cb169b

File tree

3 files changed

+21
-2
lines changed

3 files changed

+21
-2
lines changed

js/artwork.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

js/logo.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1792,6 +1792,14 @@ class Logo {
17921792
}
17931793
}
17941794
document.getElementById("stop").style.color = "white";
1795+
const saveButton = docById('saveButton');
1796+
const saveButtonAdvanced = docById('saveButtonAdvanced');
1797+
const recordButton = docById("record");
1798+
saveButton.disabled = false;
1799+
saveButtonAdvanced.disabled = false;
1800+
saveButton.className = "";
1801+
saveButtonAdvanced.className = "";
1802+
recordButton.className = "";
17951803
}
17961804
}
17971805
}

js/toolbar.js

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -355,7 +355,7 @@ class Toolbar {
355355
renderPlayIcon(onclick) {
356356
const playIcon = docById("play");
357357
const stopIcon = docById("stop");
358-
358+
const recordButton = docById("record");
359359
let isPlayIconRunning = false;
360360

361361
function handleClick() {
@@ -378,6 +378,11 @@ class Toolbar {
378378
onclick(this.activity);
379379
handleClick();
380380
stopIcon.style.color = this.stopIconColorWhenPlaying;
381+
saveButton.disabled = true;
382+
saveButtonAdvanced.disabled = true;
383+
saveButton.className = "grey-text inactiveLink";
384+
saveButtonAdvanced.className = "grey-text inactiveLink";
385+
recordButton.className = "grey-text inactiveLink";
381386
isPlayIconRunning = true;
382387
play_button_debounce_timeout = setTimeout(function() { handleClick(); }, 2000);
383388

@@ -399,9 +404,15 @@ class Toolbar {
399404
*/
400405
renderStopIcon(onclick) {
401406
const stopIcon = docById("stop");
407+
const recordButton = docById("record");
402408
stopIcon.onclick = () => {
403409
onclick(this.activity);
404410
stopIcon.style.color = "white";
411+
saveButton.disabled = false;
412+
saveButtonAdvanced.disabled = false;
413+
saveButton.className = "";
414+
saveButtonAdvanced.className = "";
415+
recordButton.className = "";
405416
};
406417
}
407418

0 commit comments

Comments
 (0)