WindowServer: Don't send menu item activations for separators

Separators can't have identifiers associated with them, so it's not possible to react meaningfully to their activation. Don't send messages about it to avoid confusing the clients.
luteresa · Nov 11, 2019 · 2fcf156 · 2fcf156
1 parent 5e61fd0
commit 2fcf156
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/Servers/WindowServer/WSMenu.cpp b/Servers/WindowServer/WSMenu.cpp
@@ -239,6 +239,9 @@ void WSMenu::clear_hovered_item()
 
 void WSMenu::did_activate(WSMenuItem& item)
 {
+ if (item.type() == WSMenuItem::Type::Separator)
+ return;
+
  if (on_item_activation)
  on_item_activation(item);