From 6b57966b58eaa8288ad76c6cf86c3f3b408cc836 Mon Sep 17 00:00:00 2001 From: Jose Diaz-Gonzalez Date: Thu, 11 Sep 2025 19:37:20 -0400 Subject: [PATCH] fix: drop importing LED settings from a theme LEDs are usually set manually by users and can even be done on a per-console or game basis, and thus do not really belong in themes. It is removed in this commit to avoid issues where users have to go back and reset their LED settings after adding a theme. --- src/internal/themes/import.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/internal/themes/import.go b/src/internal/themes/import.go index 96fb462..204226a 100644 --- a/src/internal/themes/import.go +++ b/src/internal/themes/import.go @@ -103,11 +103,11 @@ func ImportTheme(themeName string) error { } // Apply LED settings directly from manifest - if manifest.Content.Settings.LEDsIncluded { - if err := applyLEDSettings(manifest, logger); err != nil { - logger.DebugFn("Warning: Error applying LED settings: %v", err) - } - } + // if manifest.Content.Settings.LEDsIncluded { + // if err := applyLEDSettings(manifest, logger); err != nil { + // logger.DebugFn("Warning: Error applying LED settings: %v", err) + // } + // } logger.DebugFn("Theme import completed successfully: %s", themeName)