From bcd52c65cbb36ba816cfda808e197f2129ce34b5 Mon Sep 17 00:00:00 2001
From: John Scipione <jscipione@gmail.com>
Date: Thu, 31 Mar 2016 14:40:26 -0700
Subject: [PATCH] Screenshot: Don't take screenshot of open menu

...take screenshot of window with the menu open instead.

menus are windows, this is an implementation detail.
---
 src/apps/screenshot/Screenshot.cpp | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/apps/screenshot/Screenshot.cpp b/src/apps/screenshot/Screenshot.cpp
index ef179787f9..d9b4f57ad1 100644
--- a/src/apps/screenshot/Screenshot.cpp
+++ b/src/apps/screenshot/Screenshot.cpp
@@ -26,6 +26,7 @@
 #include <TranslatorFormats.h>
 
 #include <WindowInfo.h>
+#include <WindowPrivate.h>
 
 #include "Utility.h"
 
@@ -307,8 +308,10 @@ Screenshot::_GetActiveWindowFrame()
 			for (int i = 0; i < tokenCount; i++) {
 				token = tokens[i];
 				windowInfo = get_window_info(token);
-				if (windowInfo && !windowInfo->is_mini
-						&& !(windowInfo->show_hide_level > 0)) {
+				if (windowInfo != NULL
+					&& windowInfo->feel != kMenuWindowFeel
+					&& !windowInfo->is_mini
+					&& !(windowInfo->show_hide_level > 0)) {
 					foundActiveWindow = true;
 					break;
 				}