From bac8a110ecd69c542f16596a432f7a8b03ce1fb8 Mon Sep 17 00:00:00 2001
From: anntnzrb <51257127+anntnzrb@users.noreply.github.com>
Date: Wed, 15 Jun 2022 10:04:32 -0500
Subject: [PATCH] Screenshot utility improvements (maimpick) (#1114)

* maimpick delay for proper screenshots

An extra delay is needed for {window, full}-screenshots to prevent the
dmenu prompt from showing; example attached.

As of this writing I'm thinking the delay may vary depending on several
factors, to name a few... dmenu patches, compositor animations, etc.

* refactor to extract commonly used commands
---
 .local/bin/maimpick | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/.local/bin/maimpick b/.local/bin/maimpick
index 7125e61..8ea9f5e 100755
--- a/.local/bin/maimpick
+++ b/.local/bin/maimpick
@@ -4,11 +4,15 @@
 # choose the kind of screenshot to take, including copying the image or even
 # highlighting an area to copy. scrotcucks on suicidewatch right now.
 
+# variables
+output="$(date '+%y%m%d-%H%M-%S').png"
+xclip_cmd="xclip -sel clip -t image/png"
+
 case "$(printf "a selected area\\ncurrent window\\nfull screen\\na selected area (copy)\\ncurrent window (copy)\\nfull screen (copy)" | dmenu -l 6 -i -p "Screenshot which area?")" in
-	"a selected area") maim -s pic-selected-"$(date '+%y%m%d-%H%M-%S').png" ;;
-	"current window") maim -i "$(xdotool getactivewindow)" pic-window-"$(date '+%y%m%d-%H%M-%S').png" ;;
-	"full screen") maim pic-full-"$(date '+%y%m%d-%H%M-%S').png" ;;
-	"a selected area (copy)") maim -s | xclip -selection clipboard -t image/png ;;
-	"current window (copy)") maim -i "$(xdotool getactivewindow)" | xclip -selection clipboard -t image/png ;;
-	"full screen (copy)") maim | xclip -selection clipboard -t image/png ;;
+    "a selected area") maim -s pic-selected-"${output}" ;;
+    "current window") maim -q -d 0.2 -i "$(xdotool getactivewindow)" pic-window-"${output}" ;;
+    "full screen") maim -q -d 0.2 pic-full-"${output}" ;;
+    "a selected area (copy)") maim -s | ${xclip_cmd} ;;
+    "current window (copy)") maim -q -d 0.2 -i "$(xdotool getactivewindow)" | ${xclip_cmd} ;;
+    "full screen (copy)") maim -q -d 0.2 | ${xclip_cmd} ;;
 esac