From babc97500e8f52811a2efda8b73265c18eef4d0d Mon Sep 17 00:00:00 2001
From: noctuid <noct@openmailbox.org>
Date: Wed, 2 Sep 2015 13:37:03 -0400
Subject: [PATCH] Set wm var for automatic settings in single location
 (wm_autoset_for_hide_show (but not wm_autoset_for_dropdown) was taking
 advantage of the fact that wm had already been set in wm_autoset_for_all)

---
 tdrop | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/tdrop b/tdrop
index 5f7ad34..1ae565d 100755
--- a/tdrop
+++ b/tdrop
@@ -211,7 +211,6 @@ get_window_manager() {
 # can set up a rule for always floating a window or term type/class
 # alternatively, can use these settings to float just the dropdown
 wm_autoset_for_all() {
-	wm=$(get_window_manager)
 	# bspwm will use previous size when floating already
 	if [[ $wm == bspwm ]]; then
 		map_pre() {
@@ -299,8 +298,6 @@ wm_autoset_for_hide_show() {
 }
 
 wm_autoset_for_dropdown() {
-	wm=$(get_window_manager)
-
 	# tilers without floating support: can toggle fullscreen instead for the dropdown
 	if [[ $wm == herbstluftwm ]]; then
 		map_post() {
@@ -626,6 +623,7 @@ auto_show() {
 #
 
 if $auto_detect_wm; then
+	wm=$(get_window_manager)
 	wm_autoset_for_all
 	if [[ $term == auto_show ]] || [[ $term == auto_hide ]]; then
 		wm_autoset_for_hide_show