Commit 15c0cef4 authored by John Tapsell's avatar John Tapsell

Fix Perspective->Simulation menu option

There were two separate actions for the simulation view, leading to
strange behaviour when it was selected.
parent e656040b
...@@ -226,7 +226,7 @@ void MainWindow::init() ...@@ -226,7 +226,7 @@ void MainWindow::init()
// Add actions for advanced users (displayed in dropdown under "advanced") // Add actions for advanced users (displayed in dropdown under "advanced")
QList<QAction*> advancedActions; QList<QAction*> advancedActions;
advancedActions << ui.actionSimulation_View; advancedActions << ui.actionSimulationView;
advancedActions << ui.actionEngineersView; advancedActions << ui.actionEngineersView;
toolBar->setPerspectiveChangeAdvancedActions(advancedActions); toolBar->setPerspectiveChangeAdvancedActions(advancedActions);
...@@ -266,7 +266,7 @@ void MainWindow::init() ...@@ -266,7 +266,7 @@ void MainWindow::init()
apmToolBar->setFlightPlanViewAction(ui.actionMissionView); apmToolBar->setFlightPlanViewAction(ui.actionMissionView);
apmToolBar->setHardwareViewAction(ui.actionHardwareConfig); apmToolBar->setHardwareViewAction(ui.actionHardwareConfig);
apmToolBar->setSoftwareViewAction(ui.actionSoftwareConfig); apmToolBar->setSoftwareViewAction(ui.actionSoftwareConfig);
apmToolBar->setSimulationViewAction(ui.actionSimulation_View); apmToolBar->setSimulationViewAction(ui.actionSimulationView);
apmToolBar->setTerminalViewAction(ui.actionTerminalView); apmToolBar->setTerminalViewAction(ui.actionTerminalView);
QDockWidget *widget = new QDockWidget(tr("APM Tool Bar"),this); QDockWidget *widget = new QDockWidget(tr("APM Tool Bar"),this);
...@@ -1262,7 +1262,7 @@ void MainWindow::connectCommonActions() ...@@ -1262,7 +1262,7 @@ void MainWindow::connectCommonActions()
perspectives->addAction(ui.actionEngineersView); perspectives->addAction(ui.actionEngineersView);
perspectives->addAction(ui.actionMavlinkView); perspectives->addAction(ui.actionMavlinkView);
perspectives->addAction(ui.actionFlightView); perspectives->addAction(ui.actionFlightView);
perspectives->addAction(ui.actionSimulation_View); perspectives->addAction(ui.actionSimulationView);
perspectives->addAction(ui.actionMissionView); perspectives->addAction(ui.actionMissionView);
//perspectives->addAction(ui.actionConfiguration_2); //perspectives->addAction(ui.actionConfiguration_2);
perspectives->addAction(ui.actionHardwareConfig); perspectives->addAction(ui.actionHardwareConfig);
...@@ -1293,8 +1293,8 @@ void MainWindow::connectCommonActions() ...@@ -1293,8 +1293,8 @@ void MainWindow::connectCommonActions()
} }
if (currentView == VIEW_SIMULATION) if (currentView == VIEW_SIMULATION)
{ {
ui.actionSimulation_View->setChecked(true); ui.actionSimulationView->setChecked(true);
ui.actionSimulation_View->activate(QAction::Trigger); ui.actionSimulationView->activate(QAction::Trigger);
} }
if (currentView == VIEW_MISSION) if (currentView == VIEW_MISSION)
{ {
...@@ -1358,14 +1358,13 @@ void MainWindow::connectCommonActions() ...@@ -1358,14 +1358,13 @@ void MainWindow::connectCommonActions()
// Views actions // Views actions
connect(ui.actionFlightView, SIGNAL(triggered()), this, SLOT(loadPilotView())); connect(ui.actionFlightView, SIGNAL(triggered()), this, SLOT(loadPilotView()));
connect(ui.actionSimulation_View, SIGNAL(triggered()), this, SLOT(loadSimulationView())); connect(ui.actionSimulationView, SIGNAL(triggered()), this, SLOT(loadSimulationView()));
connect(ui.actionEngineersView, SIGNAL(triggered()), this, SLOT(loadEngineerView())); connect(ui.actionEngineersView, SIGNAL(triggered()), this, SLOT(loadEngineerView()));
connect(ui.actionMissionView, SIGNAL(triggered()), this, SLOT(loadOperatorView())); connect(ui.actionMissionView, SIGNAL(triggered()), this, SLOT(loadOperatorView()));
connect(ui.actionUnconnectedView, SIGNAL(triggered()), this, SLOT(loadUnconnectedView())); connect(ui.actionUnconnectedView, SIGNAL(triggered()), this, SLOT(loadUnconnectedView()));
connect(ui.actionHardwareConfig,SIGNAL(triggered()),this,SLOT(loadHardwareConfigView())); connect(ui.actionHardwareConfig,SIGNAL(triggered()),this,SLOT(loadHardwareConfigView()));
connect(ui.actionGoogleEarthView, SIGNAL(triggered()), this, SLOT(loadGoogleEarthView())); connect(ui.actionGoogleEarthView, SIGNAL(triggered()), this, SLOT(loadGoogleEarthView()));
connect(ui.actionLocal3DView, SIGNAL(triggered()), this, SLOT(loadLocal3DView())); connect(ui.actionLocal3DView, SIGNAL(triggered()), this, SLOT(loadLocal3DView()));
connect(ui.actionSimulationView, SIGNAL(triggered()), this, SLOT(loadSimulationView()));
connect(ui.actionHardwareConfig, SIGNAL(triggered()), this, SLOT(loadHardwareConfigView())); connect(ui.actionHardwareConfig, SIGNAL(triggered()), this, SLOT(loadHardwareConfigView()));
if (getCustomMode() == CUSTOM_MODE_APM) { if (getCustomMode() == CUSTOM_MODE_APM) {
...@@ -2049,7 +2048,7 @@ void MainWindow::loadSimulationView() ...@@ -2049,7 +2048,7 @@ void MainWindow::loadSimulationView()
{ {
storeViewState(); storeViewState();
currentView = VIEW_SIMULATION; currentView = VIEW_SIMULATION;
ui.actionSimulation_View->setChecked(true); ui.actionSimulationView->setChecked(true);
loadViewState(); loadViewState();
} }
} }
......
...@@ -51,7 +51,7 @@ ...@@ -51,7 +51,7 @@
<x>0</x> <x>0</x>
<y>0</y> <y>0</y>
<width>1024</width> <width>1024</width>
<height>22</height> <height>21</height>
</rect> </rect>
</property> </property>
<widget class="QMenu" name="menuMGround"> <widget class="QMenu" name="menuMGround">
...@@ -412,21 +412,6 @@ ...@@ -412,21 +412,6 @@
<string>Update the firmware of one of the connected autopilots</string> <string>Update the firmware of one of the connected autopilots</string>
</property> </property>
</action> </action>
<action name="actionSimulation_View">
<property name="checkable">
<bool>true</bool>
</property>
<property name="icon">
<iconset resource="../../qgroundcontrol.qrc">
<normaloff>:/files/images/control/launch.svg</normaloff>:/files/images/control/launch.svg</iconset>
</property>
<property name="text">
<string>Simulation</string>
</property>
<property name="toolTip">
<string>Open the simulation view</string>
</property>
</action>
<action name="actionHardwareConfig"> <action name="actionHardwareConfig">
<property name="checkable"> <property name="checkable">
<bool>true</bool> <bool>true</bool>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment