Merge pull request #819 from JaKooLit/feat/sddm-wallpaper-default-path-detection

fix(sddm): set Backgrounds/default as primary and update fallbacks;

2 files changed, 51 insertions, 29 deletions

diff --git a/config/hypr/UserScripts/WallpaperEffects.sh b/config/hypr/UserScripts/WallpaperEffects.sh
index 2ba58d0c..ac8fc0e8 100755
--- a/config/hypr/UserScripts/WallpaperEffects.sh
+++ b/config/hypr/UserScripts/WallpaperEffects.sh
@@ -108,31 +108,41 @@ main
 sleep 1
 
 if [[ -n "$choice" ]]; then
-  sddm_simple="/usr/share/sddm/themes/simple_sddm_2"
-  if [ -d "$sddm_simple" ]; then
-  
-	# Check if yad is running to avoid multiple yad notification
-	if pidof yad > /dev/null; then
-	  killall yad
-	fi
-	
-	if yad --info --text="Set current wallpaper as SDDM background?\n\nNOTE: This only applies to SIMPLE SDDM v2 Theme" \
-    --text-align=left \
-    --title="SDDM Background" \
-    --timeout=5 \
-    --timeout-indicator=right \
-    --button="yad-yes:0" \
-    --button="yad-no:1" \
-    ; then
+  # Resolve SDDM themes directory (standard and NixOS path)
+  sddm_themes_dir=""
+  if [ -d "/usr/share/sddm/themes" ]; then
+    sddm_themes_dir="/usr/share/sddm/themes"
+  elif [ -d "/run/current-system/sw/share/sddm/themes" ]; then
+    sddm_themes_dir="/run/current-system/sw/share/sddm/themes"
+  fi
 
-    # Check if terminal exists
-    if ! command -v "$terminal" &>/dev/null; then
-    notify-send -i "$iDIR/ja.png" "Missing $terminal" "Install $terminal to enable setting of wallpaper background"
-    exit 1
-    fi
+  if [ -n "$sddm_themes_dir" ]; then
+    sddm_simple="$sddm_themes_dir/simple_sddm_2"
+
+    # Only prompt if theme exists and its Backgrounds directory is writable
+    if [ -d "$sddm_simple" ] && [ -w "$sddm_simple/Backgrounds" ]; then
+      # Check if yad is running to avoid multiple yad notification
+      if pidof yad > /dev/null; then
+        killall yad
+      fi
 
-	exec $SCRIPTSDIR/sddm_wallpaper.sh --effects
-    
+      if yad --info --text="Set current wallpaper as SDDM background?\n\nNOTE: This only applies to SIMPLE SDDM v2 Theme" \
+        --text-align=left \
+        --title="SDDM Background" \
+        --timeout=5 \
+        --timeout-indicator=right \
+        --button="yad-yes:0" \
+        --button="yad-no:1" \
+        ; then
+
+        # Check if terminal exists
+        if ! command -v "$terminal" &>/dev/null; then
+          notify-send -i "$iDIR/ja.png" "Missing $terminal" "Install $terminal to enable setting of wallpaper background"
+          exit 1
+        fi
+
+        exec "$SCRIPTSDIR/sddm_wallpaper.sh" --effects
+      fi
     fi
   fi
-fi
\ No newline at end of file
+fi
diff --git a/config/hypr/UserScripts/WallpaperSelect.sh b/config/hypr/UserScripts/WallpaperSelect.sh
index a08b53ce..466832ba 100755
--- a/config/hypr/UserScripts/WallpaperSelect.sh
+++ b/config/hypr/UserScripts/WallpaperSelect.sh
@@ -101,9 +101,21 @@ menu() {
 # Offer SDDM Simple Wallpaper Option (only for non-video wallpapers)
 set_sddm_wallpaper() {
   sleep 1
-  sddm_simple="/usr/share/sddm/themes/simple_sddm_2"
 
-  if [ -d "$sddm_simple" ]; then
+  # Resolve SDDM themes directory (standard and NixOS path)
+  local sddm_themes_dir=""
+  if [ -d "/usr/share/sddm/themes" ]; then
+    sddm_themes_dir="/usr/share/sddm/themes"
+  elif [ -d "/run/current-system/sw/share/sddm/themes" ]; then
+    sddm_themes_dir="/run/current-system/sw/share/sddm/themes"
+  fi
+
+  [ -z "$sddm_themes_dir" ] && return 0
+
+  local sddm_simple="$sddm_themes_dir/simple_sddm_2"
+
+  # Only prompt if theme exists and its Backgrounds directory is writable
+  if [ -d "$sddm_simple" ] && [ -w "$sddm_simple/Backgrounds" ]; then
 
     # Check if yad is running to avoid multiple notifications
     if pidof yad >/dev/null; then
@@ -123,9 +135,9 @@ set_sddm_wallpaper() {
         notify-send -i "$iDIR/error.png" "Missing $terminal" "Install $terminal to enable setting of wallpaper background"
         exit 1
       fi
-	  
-	  exec $SCRIPTSDIR/sddm_wallpaper.sh --normal
-    
+
+      exec "$SCRIPTSDIR/sddm_wallpaper.sh" --normal
+
     fi
   fi
 }