Use actions of ViewBox's contextMenu in full menu.

The main change is on `ViewBox.getContextMenus`, which now returns an
up-to-date of actions that `GraphicsScene.addParentContextMenus` can use.

Also, `getContextMenus` was given a default implementation in the base
class (falling back on `getMenu` if defined), and some cleanup was done.

pyqtgraph/GraphicsScene/GraphicsScene.py

@@ -520,27 +520,18 @@ class GraphicsScene(QtGui.QGraphicsScene):
         ==============  ==================================================
         """
 
-        #items = self.itemsNearEvent(ev)
         menusToAdd = []
         while item is not self:
             item = item.parentItem()
-            
             if item is None:
                 item = self
+            subMenus = item.getContextMenus(event) or []
+            if isinstance(subMenus, list): ## so that some items (like FlowchartViewBox) can return multiple menus
+                menusToAdd.extend(subMenus)
+            else:
+                menusToAdd.append(subMenus)
 
-            if not hasattr(item, "getContextMenus"):
-                continue
-            
-            subMenus = item.getContextMenus(event)
-            if subMenus is None:
-                continue
-            if type(subMenus) is not list: ## so that some items (like FlowchartViewBox) can return multiple menus
-                subMenus = [subMenus]
-            
-            for sm in subMenus:
-                menusToAdd.append(sm)
-        
-        if len(menusToAdd) > 0:
+        if menusToAdd:
             menu.addSeparator()
 
         for m in menusToAdd:

pyqtgraph/flowchart/Node.py

@@ -618,9 +618,6 @@ class NodeGraphicsItem(GraphicsObject):
     def getMenu(self):
         return self.menu
     
-    def getContextMenus(self, event):
-        return [self.menu]
-    
     def raiseContextMenu(self, ev):
         menu = self.scene().addParentContextMenus(self, self.getMenu(), ev)
         pos = ev.screenPos()

pyqtgraph/flowchart/Terminal.py

@@ -437,10 +437,6 @@ class TerminalGraphicsItem(GraphicsObject):
         multi = self.menu.multiAct.isChecked()
         self.term.setMultiValue(multi)
     
-    ## probably never need this
-    #def getContextMenus(self, ev):
-        #return [self.getMenu()]
-    
     def removeSelf(self):
         self.term.node().removeTerminal(self.term)
         

pyqtgraph/graphicsItems/GraphicsItem.py

@@ -585,3 +585,6 @@ class GraphicsItem(object):
     #def update(self):
         #self._qtBaseClass.update(self)
         #print "Update:", self
+
+    def getContextMenus(self, event):
+        return [self.getMenu()] if hasattr(self, "getMenu") else []

pyqtgraph/graphicsItems/ROI.py

@@ -1203,10 +1203,6 @@ class Handle(UIGraphicsItem):
     def getMenu(self):
         return self.menu
 
-            
-    def getContextMenus(self, event):
-        return [self.menu]
-    
     def raiseContextMenu(self, ev):
         menu = self.scene().addParentContextMenus(self, self.getMenu(), ev)
         

pyqtgraph/graphicsItems/ViewBox/ViewBox.py

@@ -1064,12 +1064,7 @@ class ViewBox(GraphicsWidget):
         return self.menu
 
     def getContextMenus(self, event):
-        if self.menuEnabled():
-            return self.menu.subMenus()
-        else:
-            return None
-        #return [self.getMenu(event)]
-        
+        return self.menu.actions() if self.menuEnabled() else []
 
     def mouseDragEvent(self, ev, axis=None):
         ## if axis is specified, event will only affect that axis.

pyqtgraph/graphicsItems/ViewBox/ViewBoxMenu.py

@@ -88,22 +88,6 @@ class ViewBoxMenu(QtGui.QMenu):
         
         self.updateState()
 
-    #def copy(self):
-        #m = QtGui.QMenu()
-        #for sm in self.subMenus():
-            #if isinstance(sm, QtGui.QMenu):
-                #m.addMenu(sm)
-            #else:
-                #m.addAction(sm)
-        #m.setTitle(self.title())
-        #return m
-
-    def subMenus(self):
-        if not self.valid:
-            self.updateState()
-        return [self.viewAll] + self.axes + [self.leftMenu]
-
-
     def setExportMethods(self, methods):
         self.exportMethods = methods
         self.export.clear()