CXTEditListBox erroneous behavior?

Message Topic Search Topic Options Post Reply Create New Topic Printable Version Translate Topic

   Hi,

I believe CXTEditListBox was originally adapted from the "VC++ directories"-control found in Visual Studio. I have found a strange behavior when clicking in a non-item area. Instead of showing the "add item"-edit control, the selection is lost. This confuses users accustomed to Microsoft products.

In XTEditListBox.cpp (627) we find this:
SetCurSel(-1);
pOwner->SendMessage(WM_COMMAND, MAKEWPARAM(GetDlgCtrlID(), LBN_SELCHANGE), (LPARAM)m_hWnd);

That makes no sense. When would you want to lose the focus? It has no practical use. I think the behavior found in Visual Studio is desirable.

What do you think?

Author	Message Topic Search Topic Options Post Reply Create New Topic Printable Version Translate Topic
znakeeye Members Profile Send Private Message Find Members Posts Add to Buddy List Senior Member Joined: 26 July 2006 Status: Offline Points: 1672	Post Options Post Reply Quote znakeeye Report Post Thanks(0) Quote Reply Topic: CXTEditListBox erroneous behavior? Posted: 11 June 2007 at 6:42am
	Hi, I believe CXTEditListBox was originally adapted from the "VC++ directories"-control found in Visual Studio. I have found a strange behavior when clicking in a non-item area. Instead of showing the "add item"-edit control, the selection is lost. This confuses users accustomed to Microsoft products. In XTEditListBox.cpp (627) we find this: SetCurSel(-1); pOwner->SendMessage(WM_COMMAND, MAKEWPARAM(GetDlgCtrlID(), LBN_SELCHANGE), (LPARAM)m_hWnd); That makes no sense. When would you want to lose the focus? It has no practical use. I think the behavior found in Visual Studio is desirable. What do you think?

znakeeye Members Profile Send Private Message Find Members Posts Add to Buddy List Senior Member Joined: 26 July 2006 Status: Offline Points: 1672	Post Options Post Reply Quote znakeeye Report Post Thanks(0) Quote Reply Posted: 11 June 2007 at 6:51am
	Well, regarding the "edit item"-behavior in VS, I just realized that all you have to do is to add an extra item with no text :) Still, this deselect "feature" feels wrong.