Add "Move Tab to a New Window" in tab context menu

And open new windows in the default bounds, not in the stored bounds,
avoiding that confusing behavior when the new window is created
in the same exact position of the previous one.
The stored bounds are used for the first instance.

Author: Palollo

Explorer++/Explorer++/Explorer++.cpp

@@ -97,6 +97,7 @@ Explorerplusplus::~Explorerplusplus()
 
 HWND Explorerplusplus::CreateMainWindow(const WindowStorageData *storageData)
 {
+	bool isFirstInstance = IsFirstInstance();
 	static bool mainWindowClassRegistered = false;
 
 	if (!mainWindowClassRegistered)
@@ -119,12 +120,18 @@ HWND Explorerplusplus::CreateMainWindow(const WindowStorageData *storageData)
 
 	placement.showCmd = SW_HIDE;
 	placement.rcNormalPosition =
-		storageData ? storageData->bounds : LayoutDefaults::GetDefaultMainWindowBounds();
+		storageData && isFirstInstance ? storageData->bounds : LayoutDefaults::GetDefaultMainWindowBounds();
 	SetWindowPlacement(hwnd, &placement);
 
 	return hwnd;
 }
 
+bool Explorerplusplus::IsFirstInstance()
+{
+	HWND hPrev = FindWindow(WINDOW_CLASS_NAME, nullptr);
+	return (hPrev == nullptr);
+}
+
 ATOM Explorerplusplus::RegisterMainWindowClass(HINSTANCE instance)
 {
 	WNDCLASSEX windowClass = {};

Explorer++/Explorer++/Explorer++.h

@@ -211,6 +211,7 @@ class Explorerplusplus :
 	Explorerplusplus(App *app, const WindowStorageData *storageData);
 
 	static HWND CreateMainWindow(const WindowStorageData *storageData);
+	static bool IsFirstInstance();
 	static ATOM RegisterMainWindowClass(HINSTANCE instance);
 
 	LRESULT WindowProcedure(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam);

Explorer++/Explorer++/Explorer++.rc

@@ -443,6 +443,10 @@ void TabContainerImpl::ProcessTabCommand(UINT uMenuID, Tab &tab)
 		DuplicateTab(tab);
 		break;
 
+	case IDM_TAB_MOVETABNEWWINDOW:
+		MoveTabToNewWindow(tab);
+		break;
+
 	case IDM_TAB_OPENPARENTINNEWTAB:
 		OnOpenParentInNewTab(tab);
 		break;
@@ -1309,6 +1313,13 @@ void TabContainerImpl::DuplicateTab(const Tab &tab)
 	CreateNewTab(navigateParams, {}, &folderSettings, &folderColumns);
 }
 
+void TabContainerImpl::MoveTabToNewWindow(const Tab &tab)
+{
+	tab.GetBrowser()->OpenItem(tab.GetShellBrowserImpl()->GetDirectoryIdl().get(), 
+		OpenFolderDisposition::NewWindow);
+	CloseTab(tab);
+}
+
 int TabContainerImpl::GetDropTargetItem(const POINT &pt)
 {
 	POINT ptClient = pt;

Explorer++/Explorer++/TabContainerImpl.cpp

@@ -114,6 +114,7 @@ class TabContainerImpl : public TabContainer, public ShellDropTargetWindow<int>
 	int GetNumTabs() const;
 	int MoveTab(const Tab &tab, int newIndex);
 	void DuplicateTab(const Tab &tab);
+	void MoveTabToNewWindow(const Tab &tab);
 	bool CloseTab(const Tab &tab);
 
 	// Eventually, this should be removed.

Explorer++/Explorer++/TabContainerImpl.h

@@ -753,6 +753,7 @@
 #define IDM_VIEW_SORTBY                 40285
 #define IDM_TOOLBARS_MAINTOOLBAR        40291
 #define IDM_TOOLBARS_BOOKMARKSTOOLBAR   40292
+#define IDM_TAB_MOVETABNEWWINDOW        40316
 #define IDM_TAB_OPENPARENTINNEWTAB      40319
 #define IDM_TAB_RENAMETAB               40321
 #define IDM_VIEW_SETDEFAULTCOLUMNS      40323