From 07b526efede3c3dd5271c89a16586272442a0199 Mon Sep 17 00:00:00 2001
From: angelic-sedition <noct@openmailbox.org>
Date: Sat, 16 May 2015 02:58:47 -0400
Subject: [PATCH] Accurately get correct window id for non-terminals For some
 window managers, head -n 1 won't give the correct one

---
 tdrop | 24 ++++++++++++++++++++----
 1 file changed, 20 insertions(+), 4 deletions(-)

diff --git a/tdrop b/tdrop
index b26780e..48933cb 100755
--- a/tdrop
+++ b/tdrop
@@ -199,6 +199,10 @@ get_class_name() {
 	xprop -id "$1" WM_CLASS | awk '{gsub(/"/, ""); print $4}'
 }
 
+get_visibility() {
+	xwininfo -id "$1" | awk '/Map State/ {print $3}'
+}
+
 map_pre_command() {
 	# a user set option has higher priority
 	if [[ -n $map_pre ]]; then
@@ -268,10 +272,22 @@ win_create() {
 	$term &
 	# need to wait for window to be created
 	sleep 0.5
-	# often pids have two wids; first one has always been the right one for what I've tested
-	wid=$(xdotool search --pid $! | head -n 1)
-	echo "$wid" > /tmp/tdrop/"$term$num"
+	# often pids have two wids
+	wids=$(xdotool search --pid $!)
+	wid1=$(echo "$wids" | head -n 1)
+	wid2=$(echo "$wids" | tail -n 1)
+	if [[ -n $wid2 ]]; then
+		if [[ $(get_visibility "$wid2") == IsUnMapped ]]; then
+			wid=$wid1
+		else
+			wid=$wid2
+		fi
+	else
+		wid=$wid1
+	fi
+	# only works with pre-command
 	xdotool windowmove "$wid" "$xoff" "$yoff" windowsize "$wid" "$width" "$height"
+	echo "$wid" > /tmp/tdrop/"$term$num"
 }
 
 current_create() {
@@ -287,7 +303,7 @@ wid_toggle() {
 	wid=$(< /tmp/tdrop/"$term$num")
 	exists=true
 	if [[ -n $wid ]]; then
-		visibility=$(xwininfo -id "$wid" | awk '/Map State/ {print $3}')
+		visibility=$(get_visibility "$wid")
 		# sometimes xwininfo will still report a window as existing hence xprop check
 		if [[ -z $visibility ]] || [[ -z $(xprop -id "$wid") ]]; then
 			# window no longer exists