- MTREE_DND implemented, all drag-n-drop operations in trees went into the core;

- fixed chaos inside CCtrlBase::Subclass();
- two app dialogs switched to MTREE_DND;

git-svn-id: http://svn.miranda-ng.org/main/trunk@13566 1316c22d-e87f-b044-9b9b-93d7a3e3ba9c

2 files changed, 4 insertions, 174 deletions

diff --git a/src/modules/clist/contacts.cpp b/src/modules/clist/contacts.cpp
index 204dc3221c..98e2669fe0 100644
--- a/src/modules/clist/contacts.cpp
+++ b/src/modules/clist/contacts.cpp
@@ -376,9 +376,6 @@ static INT_PTR GetContactInfo(WPARAM, LPARAM lParam)
 

 class CContactOptsDlg : public CDlgBase

 {

-	int dragging;

-	HTREEITEM hDragItem;

-

 	CCtrlTreeView m_nameOrder;

 

 public:

@@ -386,13 +383,12 @@ public:
 		CDlgBase(hInst, IDD_OPT_CONTACT),

 		m_nameOrder(this, IDC_NAMEORDER)

 	{

-		m_nameOrder.OnBeginDrag = Callback(this, &CContactOptsDlg::OnBeginDrag);

+		m_nameOrder.SetFlags(MTREE_DND);

+		m_nameOrder.OnBeginDrag = Callback(this, &CContactOptsDlg::OnBeginDrag);
 	}

 

 	virtual void OnInitDialog()

 	{

-		dragging = 0;

-

 		TVINSERTSTRUCT tvis;

 		tvis.hParent = NULL;

 		tvis.hInsertAfter = TVI_LAST;

@@ -423,82 +419,7 @@ public:
 	{

 		LPNMTREEVIEW pNotify = evt->nmtv;

 		if (pNotify->itemNew.lParam == 0 || pNotify->itemNew.lParam == SIZEOF(nameOrderDescr) - 1)

-			return;

-		

-		SetCapture(m_hwnd);

-		dragging = 1;

-		hDragItem = pNotify->itemNew.hItem;

-		m_nameOrder.SelectItem(hDragItem);

-	}

-

-	virtual INT_PTR DlgProc(UINT msg, WPARAM wParam, LPARAM lParam)

-	{

-		TVHITTESTINFO hti;

-

-		switch (msg) {

-		case WM_MOUSEMOVE:

-			if (!dragging)

-				break;

-

-			hti.pt.x = (short)LOWORD(lParam);

-			hti.pt.y = (short)HIWORD(lParam);

-			ClientToScreen(m_hwnd, &hti.pt);

-			ScreenToClient(m_nameOrder.GetHwnd(), &hti.pt);

-			m_nameOrder.HitTest(&hti);

-			if (hti.flags & (TVHT_ONITEM | TVHT_ONITEMRIGHT)) {

-				hti.pt.y -= m_nameOrder.GetItemHeight() / 2;

-				m_nameOrder.HitTest(&hti);

-				m_nameOrder.SetInsertMark(hti.hItem, 1);

-			}

-			else {

-				if (hti.flags & TVHT_ABOVE) m_nameOrder.SendMsg(WM_VSCROLL, MAKEWPARAM(SB_LINEUP, 0), 0);

-				if (hti.flags & TVHT_BELOW) m_nameOrder.SendMsg(WM_VSCROLL, MAKEWPARAM(SB_LINEDOWN, 0), 0);

-				m_nameOrder.SetInsertMark(NULL, 0);

-			}

-			break;

-

-		case WM_LBUTTONUP:

-			if (!dragging)

-				break;

-

-			m_nameOrder.SetInsertMark(NULL, 0);

-			dragging = 0;

-			ReleaseCapture();

-

-			hti.pt.x = (short)LOWORD(lParam);

-			hti.pt.y = (short)HIWORD(lParam);

-			ClientToScreen(m_hwnd, &hti.pt);

-			ScreenToClient(m_nameOrder.GetHwnd(), &hti.pt);

-			hti.pt.y -= m_nameOrder.GetItemHeight() / 2;

-			m_nameOrder.HitTest(&hti);

-			if (hDragItem == hti.hItem)

-				break;

-

-			TVITEMEX tvi;

-			tvi.mask = TVIF_HANDLE | TVIF_PARAM;

-			tvi.hItem = hti.hItem;

-			m_nameOrder.GetItem(&tvi);

-			if (tvi.lParam == SIZEOF(nameOrderDescr) - 1)

-				break;

-

-			if (hti.flags & (TVHT_ONITEM | TVHT_ONITEMRIGHT)) {

-				TCHAR name[128];

-				TVINSERTSTRUCT tvis = { 0 };

-				tvis.itemex.mask = TVIF_HANDLE | TVIF_PARAM | TVIF_TEXT | TVIF_PARAM;

-				tvis.itemex.stateMask = 0xFFFFFFFF;

-				tvis.itemex.pszText = name;

-				tvis.itemex.cchTextMax = SIZEOF(name);

-				tvis.itemex.hItem = hDragItem;

-				m_nameOrder.GetItem(&tvis.itemex);

-				m_nameOrder.DeleteItem(hDragItem);

-				tvis.hParent = NULL;

-				tvis.hInsertAfter = hti.hItem;

-				m_nameOrder.SelectItem(m_nameOrder.InsertItem(&tvis));

-				NotifyChange();

-			}

-			break;

-		}

-		return CDlgBase::DlgProc(msg, wParam, lParam);

+			pNotify->hdr.code = 0; // deny dragging

 	}

 };

 

diff --git a/src/modules/clist/protocolorder.cpp b/src/modules/clist/protocolorder.cpp
index a454793de3..274ae0dce6 100644
--- a/src/modules/clist/protocolorder.cpp
+++ b/src/modules/clist/protocolorder.cpp
@@ -149,8 +149,7 @@ public:
 	{

 		m_btnReset.OnClick = Callback(this, &CProtocolOrderOpts::onReset_Click);

 

-		m_order.SetFlags(MTREE_CHECKBOX);

-		m_order.OnBeginDrag = Callback(this, &CProtocolOrderOpts::onOrder_BeginDrag);

+		m_order.SetFlags(MTREE_CHECKBOX | MTREE_DND);

 		m_order.OnDeleteItem = Callback(this, &CProtocolOrderOpts::onOrder_DeleteItem);

 	}

 

@@ -216,96 +215,6 @@ public:
 		if (pnmtv)

 			mir_free((ProtocolData*)pnmtv->itemOld.lParam);

 	}

-

-	void onOrder_BeginDrag(CCtrlTreeView::TEventInfo *env)

-	{

-		SetCapture(m_hwnd);

-		m_bDragging = true;

-		m_hDragItem = env->nmtv->itemNew.hItem;

-		m_order.SelectItem(m_hDragItem);

-	}

-

-	virtual INT_PTR DlgProc(UINT msg, WPARAM wParam, LPARAM lParam)

-	{

-		TVHITTESTINFO hti;

-

-		switch (msg) {

-		case WM_MOUSEMOVE:

-			if (!m_bDragging)

-				break;

-

-			hti.pt.x = (short)LOWORD(lParam);

-			hti.pt.y = (short)HIWORD(lParam);

-			ClientToScreen(m_hwnd, &hti.pt);

-			ScreenToClient(m_order.GetHwnd(), &hti.pt);

-			m_order.HitTest(&hti);

-			if (hti.flags & (TVHT_ONITEM | TVHT_ONITEMRIGHT)) {

-				HTREEITEM it = hti.hItem;

-				hti.pt.y -= m_order.GetItemHeight() / 2;

-				m_order.HitTest(&hti);

-				if (!(hti.flags & TVHT_ABOVE))

-					m_order.SetInsertMark(hti.hItem, 1);

-				else

-					m_order.SetInsertMark(it, 0);

-			}

-			else {

-				if (hti.flags & TVHT_ABOVE) m_order.SendMsg(WM_VSCROLL, MAKEWPARAM(SB_LINEUP, 0), 0);

-				if (hti.flags & TVHT_BELOW) m_order.SendMsg(WM_VSCROLL, MAKEWPARAM(SB_LINEDOWN, 0), 0);

-				m_order.SetInsertMark(NULL, 0);

-			}

-			break;

-

-		case WM_LBUTTONUP:

-			if (!m_bDragging)

-				break;

-

-			m_order.SetInsertMark(NULL, 0);

-			m_bDragging = false;

-			ReleaseCapture();

-

-			hti.pt.x = (short)LOWORD(lParam);

-			hti.pt.y = (short)HIWORD(lParam);

-			ClientToScreen(m_hwnd, &hti.pt);

-			ScreenToClient(m_order.GetHwnd(), &hti.pt);

-			hti.pt.y -= m_order.GetItemHeight() / 2;

-			m_order.HitTest(&hti);

-			if (m_hDragItem == hti.hItem)

-				break;

-			if (hti.flags & TVHT_ABOVE)

-				hti.hItem = TVI_FIRST;

-

-			TVITEMEX tvi;

-			tvi.mask = TVIF_HANDLE | TVIF_PARAM | TVIF_IMAGE;

-			tvi.hItem = m_hDragItem;

-			m_order.GetItem(&tvi);

-			if (hti.flags & (TVHT_ONITEM | TVHT_ONITEMRIGHT) || (hti.hItem == TVI_FIRST)) {

-				TCHAR name[128];

-				TVINSERTSTRUCT tvis;

-				tvis.itemex.mask = TVIF_HANDLE | TVIF_PARAM | TVIF_TEXT | TVIF_IMAGE | TVIF_SELECTEDIMAGE;

-				tvis.itemex.stateMask = 0xFFFFFFFF;

-				tvis.itemex.pszText = name;

-				tvis.itemex.cchTextMax = SIZEOF(name);

-				tvis.itemex.hItem = m_hDragItem;

-				tvis.itemex.iImage = tvis.itemex.iSelectedImage = tvi.iImage;

-				m_order.GetItem(&tvis.itemex);

-

-				// the pointed lParam will be freed inside TVN_DELETEITEM

-				// so lets substitute it with 0

-				ProtocolData *lpOldData = (ProtocolData*)tvis.itemex.lParam;

-				tvis.itemex.lParam = 0;

-				m_order.SetItem(&tvis.itemex);

-

-				// now current item contain lParam = 0 we can delete it. the memory will be kept.

-				tvis.itemex.lParam = (LPARAM)lpOldData;

-				m_order.DeleteItem(m_hDragItem);

-				tvis.hParent = NULL;

-				tvis.hInsertAfter = hti.hItem;

-				m_order.SelectItem(m_order.InsertItem(&tvis));

-				NotifyChange();

-			}

-		}

-		return CDlgBase::DlgProc(msg, wParam, lParam);

-	}

 };

 

 int ProtocolOrderOptInit(WPARAM wParam, LPARAM)