From bc6d1b5304f715ad0d8be27efd6630f820572da4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Petr=20Mr=C3=A1zek?= <peterix@gmail.com>
Date: Sat, 6 Aug 2016 15:39:29 +0200
Subject: GH-338, GH-513, GH-700 Unify edit instance with console window

* The resulting instance window can be closed at any point.
* Main window is kept open and running instances are marked with a badge.
* Multiple instances can now run from the same MultiMC - it's even more **multi** now.
* MultiMC can be entirely closed, keeping Minecraft(s) running.
---
 application/MainWindow.cpp | 66 +++++++++++++++++++++++++++++++++++++---------
 1 file changed, 54 insertions(+), 12 deletions(-)

(limited to 'application/MainWindow.cpp')

diff --git a/application/MainWindow.cpp b/application/MainWindow.cpp
index 06d165da..88c0fc09 100644
--- a/application/MainWindow.cpp
+++ b/application/MainWindow.cpp
@@ -67,6 +67,7 @@
 #include <updater/UpdateChecker.h>
 #include <DesktopServices.h>
 
+#include "InstanceWindow.h"
 #include "InstancePageProvider.h"
 #include "InstanceProxyModel.h"
 #include "JavaCommon.h"
@@ -1424,24 +1425,62 @@ void MainWindow::on_actionSettings_triggered()
 	update();
 }
 
+InstanceWindow *MainWindow::showInstanceWindow(InstancePtr instance, QString page)
+{
+	if(!instance)
+		return nullptr;
+	auto id = instance->id();
+	InstanceWindow * window = nullptr;
+
+	auto iter = m_instanceWindows.find(id);
+	if(iter != m_instanceWindows.end())
+	{
+		window = *iter;
+		window->raise();
+		window->activateWindow();
+	}
+	else
+	{
+		window = new InstanceWindow(instance, this);
+		m_instanceWindows[id] = window;
+		connect(window, &InstanceWindow::isClosing, this, &MainWindow::on_instanceWindowClose);
+	}
+	if(!page.isEmpty())
+	{
+		window->selectPage(page);
+	}
+	return window;
+}
+
+void MainWindow::on_instanceWindowClose()
+{
+	auto senderWindow = qobject_cast<InstanceWindow *>(QObject::sender());
+	if(!senderWindow)
+	{
+		return;
+	}
+	m_instanceWindows.remove(senderWindow->instanceId());
+}
+
+
 void MainWindow::on_actionInstanceSettings_triggered()
 {
-	SettingsUI::ShowInstancePageDialog(m_selectedInstance, this, "settings");
+	showInstanceWindow(m_selectedInstance, "settings");
 }
 
 void MainWindow::on_actionEditInstNotes_triggered()
 {
-	SettingsUI::ShowInstancePageDialog(m_selectedInstance, this, "notes");
+	showInstanceWindow(m_selectedInstance, "notes");
 }
 
 void MainWindow::on_actionEditInstance_triggered()
 {
-	SettingsUI::ShowInstancePageDialog(m_selectedInstance, this);
+	showInstanceWindow(m_selectedInstance);
 }
 
 void MainWindow::on_actionScreenshots_triggered()
 {
-	SettingsUI::ShowInstancePageDialog(m_selectedInstance, this, "screenshots");
+	showInstanceWindow(m_selectedInstance, "screenshots");
 }
 
 void MainWindow::on_actionManageAccounts_triggered()
@@ -1586,16 +1625,19 @@ void MainWindow::on_actionLaunchInstanceOffline_triggered()
 
 void MainWindow::launch(InstancePtr instance, bool online, BaseProfilerFactory *profiler)
 {
-	if(!instance->canLaunch())
+	if(instance->canLaunch())
 	{
-		return;
+		m_launchController.reset(new LaunchController());
+		m_launchController->setInstance(instance);
+		m_launchController->setOnline(online);
+		m_launchController->setParentWidget(this);
+		m_launchController->setProfiler(profiler);
+		m_launchController->start();
+	}
+	else if (instance->isRunning())
+	{
+		showInstanceWindow(instance, "console");
 	}
-	m_launchController.reset(new LaunchController());
-	m_launchController->setInstance(instance);
-	m_launchController->setOnline(online);
-	m_launchController->setParentWidget(this);
-	m_launchController->setProfiler(profiler);
-	m_launchController->start();
 }
 
 void MainWindow::taskEnd()
-- 
cgit v1.2.3