From e1b33fbc405cb6edddbe45b54cbc5bc0b9428fd5 Mon Sep 17 00:00:00 2001
From: John Preston <johnprestonmail@gmail.com>
Date: Fri, 1 Nov 2024 14:35:33 +0400
Subject: [PATCH] Remove unneeded separator from context menu.

---
 Telegram/SourceFiles/history/history_inner_widget.cpp       | 3 ---
 .../SourceFiles/history/view/history_view_context_menu.cpp  | 6 +++---
 2 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/Telegram/SourceFiles/history/history_inner_widget.cpp b/Telegram/SourceFiles/history/history_inner_widget.cpp
index d026cf9ca..238faaccb 100644
--- a/Telegram/SourceFiles/history/history_inner_widget.cpp
+++ b/Telegram/SourceFiles/history/history_inner_widget.cpp
@@ -2902,9 +2902,6 @@ void HistoryInner::showContextMenu(QContextMenuEvent *e, bool showFromTouch) {
 			HistoryView::EmojiPacksSource::Message,
 			_controller);
 		const auto added = (_menu->actions().size() > wasAmount);
-		if (!added && !_menu->empty()) {
-			_menu->addSeparator();
-		}
 		HistoryView::AddSelectRestrictionAction(
 			_menu,
 			textItem ? textItem : _dragStateItem,
diff --git a/Telegram/SourceFiles/history/view/history_view_context_menu.cpp b/Telegram/SourceFiles/history/view/history_view_context_menu.cpp
index a9c699188..180e44cc6 100644
--- a/Telegram/SourceFiles/history/view/history_view_context_menu.cpp
+++ b/Telegram/SourceFiles/history/view/history_view_context_menu.cpp
@@ -1280,9 +1280,6 @@ base::unique_qptr<Ui::PopupMenu> FillContextMenu(
 	}
 	if (item) {
 		const auto added = (result->actions().size() > wasAmount);
-		if (!added) {
-			result->addSeparator();
-		}
 		AddSelectRestrictionAction(result, item, !added);
 	}
 	if (hasWhoReactedItem) {
@@ -1885,6 +1882,9 @@ void AddSelectRestrictionAction(
 		|| item->isSponsored()) {
 		return;
 	}
+	if (addIcon && !menu->empty()) {
+		menu->addSeparator();
+	}
 	auto button = base::make_unique_q<Ui::Menu::MultilineAction>(
 		menu->menu(),
 		menu->st().menu,