From 83ea4af492d1a08d825b19c3e5026d99a189e083 Mon Sep 17 00:00:00 2001
From: John Scipione <jscipione@gmail.com>
Date: Tue, 20 Aug 2013 13:34:14 -0400
Subject: [PATCH] BMenu: Don't hit test separator items

Fixes #4147 while not causing the regression described in #6610
like hrev35248 did.
---
 src/kits/interface/Menu.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/kits/interface/Menu.cpp b/src/kits/interface/Menu.cpp
index a67b5c4efc..565a77e954 100644
--- a/src/kits/interface/Menu.cpp
+++ b/src/kits/interface/Menu.cpp
@@ -2528,8 +2528,10 @@ BMenu::_HitTestItems(BPoint where, BPoint slop) const
 	int32 itemCount = CountItems();
 	for (int32 i = 0; i < itemCount; i++) {
 		BMenuItem* item = ItemAt(i);
-		if (item->Frame().Contains(where))
+		if (item->Frame().Contains(where)
+			&& dynamic_cast<BSeparatorItem*>(item) == NULL) {
 			return item;
+		}
 	}
 
 	return NULL;