diff options
author | Valentin Fokin <fokinv@inf.u-szeged.hu> | 2018-03-27 15:28:11 +0200 |
---|---|---|
committer | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2018-04-05 21:33:11 +0000 |
commit | 51e0e0433f94d0190c510efcea087c021cc6fc5d (patch) | |
tree | ea6a9f9ac1e43d5c793a9f65b04c19a02c525008 /src/webenginewidgets | |
parent | 7d545f6bb6c4bf698410fe4b8f8628cd9715ab0b (diff) |
Remove an unnecessary ContextMenu Separator
When right clicking on a simple image there would be a separator on the
top of the DefaultContextMenu. This patch fixes it by checking if the
context menu has any elements in it before adding a separator.
Change-Id: I6bbe91fdac9f49006be8cc73df1d97d34f3c2848
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'src/webenginewidgets')
-rw-r--r-- | src/webenginewidgets/api/qwebenginepage.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/webenginewidgets/api/qwebenginepage.cpp b/src/webenginewidgets/api/qwebenginepage.cpp index 20a0297cd..6b91bd221 100644 --- a/src/webenginewidgets/api/qwebenginepage.cpp +++ b/src/webenginewidgets/api/qwebenginepage.cpp @@ -2350,7 +2350,8 @@ void QContextMenuBuilder::addMenuItem(ContextMenuItem menuItem) } return; case ContextMenuItem::Separator: - m_menu->addSeparator(); + if (!m_menu->isEmpty()) + m_menu->addSeparator(); return; } action->setEnabled(isMenuItemEnabled(menuItem)); |