From be39ceb803d1c9256ff9c10e5f39e52e760d20d9 Mon Sep 17 00:00:00 2001 From: keelus Date: Sun, 30 Jul 2023 02:17:34 +0200 Subject: [PATCH] :bug: Deleting first activated color will properly activate the second one --- web/static/js/panels/LedControl.js | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/web/static/js/panels/LedControl.js b/web/static/js/panels/LedControl.js index 21076c0..2a2d471 100644 --- a/web/static/js/panels/LedControl.js +++ b/web/static/js/panels/LedControl.js @@ -102,13 +102,15 @@ $(".color .delete").on("click", (e) => { }, success: function (r) { let colorParent = $(e.target).closest(".color") + if ($(colorParent).hasClass("selected")) { // If deleted is the activated $(".color.selected").removeClass("selected") + $(colorParent).remove() $($(`[mode='${mode}'] .color`)[0]).addClass("selected") + } else { + $(colorParent).remove() } - - $(colorParent).remove() showPopup(`Color removed.`, 3000, "success") @@ -130,14 +132,16 @@ $(".gradient .delete").on("click", (e) => { }, success: function (r) { let gradientParent = $(e.target).closest(".gradient") + if ($(gradientParent).hasClass("selected")) { // If deleted is the activated $(".gradient.selected").removeClass("selected") + $(gradientParent).remove() $($(".gradient")[0]).addClass("selected") + } else { + $(gradientParent).remove() } - $(gradientParent).remove() - showPopup(`Gradient deleted.`, 3000, "success") }, error: function(r) {