Merge pull request #361 from dagoodma/map_fixes

Map fixes

src/ui/map/QGCMapToolBar.cc

@@ -60,18 +60,27 @@ void QGCMapToolBar::setMap(QGCMapWidget* map)
         action =  mapTypesMenu.addAction(tr("Bing Hybrid"),this,SLOT(setMapType()));
         action->setData(MapType::BingHybrid);
         action->setCheckable(true);
+#ifdef MAP_DEFAULT_TYPE_BING
+        action->setChecked(true);
+#endif
         mapTypesGroup->addAction(action);
 
         action =  mapTypesMenu.addAction(tr("Google Hybrid"),this,SLOT(setMapType()));
         action->setData(MapType::GoogleHybrid);
         action->setCheckable(true);
+#ifdef MAP_DEFAULT_TYPE_GOOGLE
+        action->setChecked(true);
+#endif
         mapTypesGroup->addAction(action);
 
         action =  mapTypesMenu.addAction(tr("OpenStreetMap"),this,SLOT(setMapType()));
         action->setData(MapType::OpenStreetMap);
         action->setCheckable(true);
+#ifdef MAP_DEFAULT_TYPE_OSM
+        action->setChecked(true);
+#endif
         mapTypesGroup->addAction(action);
-        //TODO check current item
+
         optionsMenu.addMenu(&mapTypesMenu);
 
 

src/ui/map/QGCMapWidget.cc

@@ -34,7 +34,13 @@ QGCMapWidget::QGCMapWidget(QWidget *parent) :
     //this->SetShowTileGridLines(true);
 
     //default appears to be Google Hybrid, and is broken currently
+#if defined MAP_DEFAULT_TYPE_BING
     this->SetMapType(MapType::BingHybrid);
+#elif defined MAP_DEFAULT_TYPE_GOOGLE
+    this->SetMapType(MapType::GoogleHybrid);
+#else
+    this->SetMapType(MapType::OpenStreetMap);
+#endif
 
     this->setContextMenuPolicy(Qt::ActionsContextMenu);
 
@@ -75,7 +81,7 @@ void QGCMapWidget::guidedActionTriggered()
             }
         }
         // Create new waypoint and send it to the WPManager to send out.
-        internals::PointLatLng pos = map->FromLocalToLatLng(mousePressPos.x(), mousePressPos.y());
+        internals::PointLatLng pos = map->FromLocalToLatLng(contextMousePressPos.x(), contextMousePressPos.y());
         qDebug() << "Guided action requested. Lat:" << pos.Lat() << "Lon:" << pos.Lng();
         Waypoint wp;
         wp.setLatitude(pos.Lat());
@@ -113,7 +119,7 @@ void QGCMapWidget::cameraActionTriggered()
     if (newmav)
     {
         newmav->setMountConfigure(4,true,true,true);
-        internals::PointLatLng pos = map->FromLocalToLatLng(mousePressPos.x(), mousePressPos.y());
+        internals::PointLatLng pos = map->FromLocalToLatLng(contextMousePressPos.x(), contextMousePressPos.y());
         newmav->setMountControl(pos.Lat(),pos.Lng(),100,true);
     }
 }
@@ -165,15 +171,6 @@ void QGCMapWidget::mouseReleaseEvent(QMouseEvent *event)
     mapcontrol::OPMapWidget::mouseReleaseEvent(event);
 }
 
-/*
-void QGCMapWidget::contextMenuEvent(QContextMenuEvent *event)
-{
-    // TODO Remove this method
-    qDebug() << "Context menu event triggered.";
-    mapcontrol::OPMapWidget::contextMenuEvent(event);
-}
-*/
-
 QGCMapWidget::~QGCMapWidget()
 {
     SetShowHome(false);	// doing this appears to stop the map lib crashing on exit

src/ui/map/QGCMapWidget.h

@@ -5,6 +5,11 @@
 #include <QTimer>
 #include "../../../libs/opmapcontrol/opmapcontrol.h"
 
+// Choose one default map type
+#define MAP_DEFAULT_TYPE_BING
+//#define MAP_DEFAULT_TYPE_GOOGLE
+//#define MAP_DEFAULT_TYPE_OSM
+
 class UASInterface;
 class UASWaypointManager;
 class Waypoint;