- fix to enable custom map canvas item positioning

- QAction of map tool now gets unchecked when tool has been deactivated
  even if it's in another action group


git-svn-id: http://svn.osgeo.org/qgis/trunk@5154 c8812cc2-4d05-0410-92ff-de0c093fc19c

Author: wonder

src/gui/qgsmapcanvasitem.h

@@ -57,15 +57,11 @@ class QgsMapCanvasItem : public QObject, public Q3CanvasRectangle
 
     //! transformation from map coordinates to screen coordinates
     QPoint toCanvasCoords(const QgsPoint& point);
-    
-    
-    
-  public slots:
-    
+
     /** called on changed extents or changed item rectangle
      * Override this in your subclass if you wish to have custom
      * behaviour for when the canvas area of interest is changed */
-    void updatePosition();
+    virtual void updatePosition();
 
   protected:
 

src/gui/qgsmaptool.cpp

@@ -48,13 +48,15 @@ void QgsMapTool::activate()
 {
   // make action active
   if (mAction)
-    mAction->setOn(true);
+    mAction->setChecked(true);
 
-  // set cursor (map tools usually set it in constructor
+  // set cursor (map tools usually set it in constructor)
   mCanvas->setCursor(mCursor);
 }
 
 
 void QgsMapTool::deactivate()
 {
+  if (mAction)
+    mAction->setChecked(false);
 }