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/webengine | |
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/webengine')
-rw-r--r-- | src/webengine/ui_delegates_manager.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/webengine/ui_delegates_manager.cpp b/src/webengine/ui_delegates_manager.cpp index 12474a1f1..573d5018b 100644 --- a/src/webengine/ui_delegates_manager.cpp +++ b/src/webengine/ui_delegates_manager.cpp @@ -238,7 +238,7 @@ void UIDelegatesManager::addMenuSeparator(QObject *menu) sep->setParent(menu); QQmlListReference entries(menu, defaultPropertyName(menu), qmlEngine(m_view)); - if (entries.isValid()) + if (entries.isValid() && entries.count() > 0) entries.append(sep); } |