MenuBar. Menu, MenuItem: Update documentation to new API

Also removed Menu.visibleParent since it had no apparent use. Likewise,
trigger(), and the triggered() signal are now only in MenuItem.

Change-Id: Ide194505664c60906019422a503b189250d6a86e
Reviewed-by: Jens Bache-Wiig <jens.bache-wiig@digia.com>

1 files changed, 8 insertions, 6 deletions

diff --git a/src/controls/Menu.qml b/src/controls/Menu.qml
index 3f258fa9..7ba8a41e 100644
--- a/src/controls/Menu.qml
+++ b/src/controls/Menu.qml
@@ -84,6 +84,11 @@ import "Styles/Settings.js" as Settings
     }
     \endcode
 
+    Note that some properties, like \c enabled, \c text, or \c iconSource, only make sense
+    depending on the context in which the menu is being used. The three main use cases are
+    as a \l MenuBar top-level menu, as a submenu inside another menu, or as a standalone,
+    or context, menu.
+
     \sa MenuBar, MenuItem, MenuSeparator
 */
 MenuPrivate {
@@ -182,8 +187,9 @@ MenuPrivate {
         function triggerAndDismiss() {
             var item = itemsRepeater.itemAt(root.__currentIndex)
             if (item && !item.isSeparator) {
-                item.menuItem.trigger()
                 root.__dismissMenu()
+                if (!item.hasSubmenu)
+                    item.menuItem.trigger()
             }
         }
 
@@ -269,11 +275,7 @@ MenuPrivate {
                             }
                         }
 
-                        Component.onCompleted: {
-                            menuItem.__visualItem = menuItemLoader
-                            if (hasSubmenu)
-                                menuItem.visualParent = menuItemLoader
-                        }
+                        Component.onCompleted: menuItem.__visualItem = menuItemLoader
                     }
                 }