Update setwall syntax & more

README.md

@@ -16,6 +16,8 @@ Tested working on Arch with:
 - Awesome
 - dwm
 
+I (Flock137) have also tested on Debian, it works well with xfce4. It is expected that it should also work on other xfce4.
+
 Not tested on other distros and BSPWM
 
 ##### Change wallpaper quickly using setwall script
@@ -174,11 +176,19 @@ X-GNOME-Autostart-enabled=true
 Comment=Script to set wallpaper
 ```
 
+# Desktop Launcher 
+
+Copy the `setwall.desktop` to your executable path, either in local (`~/.local/share/applications`) or system-wide (`~/usr/share/applications`). 
+
+It's more recommended that you put it in local
+
 ---
 
 #### RAM consumption
 
-Base on my tests, mplayer only takes up less tha 300MB RAM. It should not cause significant slowdowns on your system.
+Base on my tests, mplayer only takes up less than 300MB RAM. It should not cause significant slowdowns on your system. 
+
+However, for machines with no dedicated GPU, be expected that your laptop will be heated up a bit. Otherwise, do consider to let your GPU handle your wallpaper within some forms or manner. 
 
 #### Why not mpv?
 

scripts/setwall

@@ -1,4 +1,4 @@
-#!/bin/sh
+#!/bin/bash
 
 # Use xwinwrap-git from AUR (https://github.com/takase1121/xwinwrap)
 
@@ -45,7 +45,7 @@ function setwall() {
             # Use gifview to play gif if current wallpaper is .gif
             sleep 0.01
             printf "$GREEN[INFO]$NC Setting wallpaper to $var.gif\n"
-            xwinwrap $args -- gifview -w %WID $HOME/.config/video-wallpapers/wallpapers/$var.gif --geometry 2560x1440 -a >/dev/null &
+            xwinwrap $args -- gifview -wid WID $HOME/.config/video-wallpapers/wallpapers/$var.gif -g 1920x1200 -a >/dev/null &
         fi
     fi
 }
@@ -61,11 +61,15 @@ else
 fi
 
 if [[ $VIDEOPLAYER == "mplayer" ]]; then
-    videoargs="-wid %WID -quiet -nosound $HOME/.config/video-wallpapers/wallpapers/$var.mp4 -loop 0 -nolirc -nomouseinput"
+    videoargs="-wid WID -quiet -nosound -x 1920 -y 1200  $HOME/.config/video-wallpapers/wallpapers/$var.mp4 -loop 0 -nolirc -nomouseinput"
 else
-    videoargs="-wid %WID -quiet --no-audio $HOME/.config/video-wallpapers/wallpapers/$var.mp4 -loop"
+    videoargs="-wid WID -quiet --no-audio -x 1920 -y 1200 $HOME/.config/video-wallpapers/wallpapers/$var.mp4 -loop"
 fi
 
+# You might need to fix the geometry to fit the video to the screen
+# By using -x <width> -y <height>
+# -zoom didnt work for my case, but you might like to try that
+
 setwall
 
 #else if [[ $(ls $HOME/.config/video-wallpapers/wallpapers/$var | grep .html) ]] # implement html wallpaper

setwall.desktop

@@ -0,0 +1,5 @@
+[Desktop Entry]
+Type=Application
+Name=setwall
+Exec=$HOME/.config/video-wallpapers/setwall
+Comment=Script to set wallpaper