#endif
#include "wx/clipbrd.h"
-#include "wx/dynlib.h"
#include "wx/wupdlock.h"
#include "wx/msw/private.h"
// implementation
// ============================================================================
+namespace
+{
+
+// Check if the given message should be forwarded from the edit control which
+// is part of the combobox to wxComboBox itself. All messages generating the
+// events that the code using wxComboBox could be interested in must be
+// forwarded.
+bool ShouldForwardFromEditToCombo(UINT message)
+{
+ switch ( message )
+ {
+ case WM_KEYUP:
+ case WM_KEYDOWN:
+ case WM_CHAR:
+ case WM_SYSCHAR:
+ case WM_SYSKEYDOWN:
+ case WM_SYSKEYUP:
+ case WM_SETFOCUS:
+ case WM_KILLFOCUS:
+ case WM_CUT:
+ case WM_COPY:
+ case WM_PASTE:
+ return true;
+ }
+
+ return false;
+}
+
+} // anonymous namespace
+
// ----------------------------------------------------------------------------
// wnd proc for subclassed edit control
// ----------------------------------------------------------------------------
HWND hwndCombo = ::GetParent(hWnd);
wxWindow *win = wxFindWinFromHandle((WXHWND)hwndCombo);
- switch ( message )
+ if ( ShouldForwardFromEditToCombo(message) )
{
- // forward some messages to the combobox to generate the appropriate
- // wxEvents from them
- case WM_KEYUP:
- case WM_KEYDOWN:
- case WM_CHAR:
- case WM_SYSCHAR:
- case WM_SYSKEYDOWN:
- case WM_SYSKEYUP:
- case WM_SETFOCUS:
- case WM_KILLFOCUS:
+ wxComboBox *combo = wxDynamicCast(win, wxComboBox);
+ if ( !combo )
+ {
+ // we can get WM_KILLFOCUS while our parent is already half
+ // destroyed and hence doesn't look like a combobx any
+ // longer, check for it to avoid bogus assert failures
+ if ( !win->IsBeingDeleted() )
{
- wxComboBox *combo = wxDynamicCast(win, wxComboBox);
- if ( !combo )
- {
- // we can get WM_KILLFOCUS while our parent is already half
- // destroyed and hence doesn't look like a combobx any
- // longer, check for it to avoid bogus assert failures
- if ( !win->IsBeingDeleted() )
- {
- wxFAIL_MSG( wxT("should have combo as parent") );
- }
- }
- else if ( combo->MSWProcessEditMsg(message, wParam, lParam) )
- {
- // handled by parent
- return 0;
- }
+ wxFAIL_MSG( wxT("should have combo as parent") );
}
- break;
-
- case WM_GETDLGCODE:
- {
- wxCHECK_MSG( win, 0, wxT("should have a parent") );
+ }
+ else if ( combo->MSWProcessEditMsg(message, wParam, lParam) )
+ {
+ // handled by parent
+ return 0;
+ }
+ }
+ else if ( message == WM_GETDLGCODE )
+ {
+ wxCHECK_MSG( win, 0, wxT("should have a parent") );
- if ( win->GetWindowStyle() & wxTE_PROCESS_ENTER )
- {
- // need to return a custom dlg code or we'll never get it
- return DLGC_WANTMESSAGE;
- }
- }
- break;
+ if ( win->GetWindowStyle() & wxTE_PROCESS_ENTER )
+ {
+ // need to return a custom dlg code or we'll never get it
+ return DLGC_WANTMESSAGE;
+ }
}
return ::CallWindowProc(CASTWNDPROC gs_wndprocEdit, hWnd, message, wParam, lParam);
return true;
}
}
- // fall through
-
- case WM_SYSCHAR:
- return HandleChar(wParam, lParam);
-
- case WM_SYSKEYDOWN:
- case WM_KEYDOWN:
- return HandleKeyDown(wParam, lParam);
-
- case WM_SYSKEYUP:
- case WM_KEYUP:
- return HandleKeyUp(wParam, lParam);
+ // fall through, WM_CHAR is one of the message we should forward.
- case WM_SETFOCUS:
- return HandleSetFocus((WXHWND)wParam);
-
- case WM_KILLFOCUS:
- return HandleKillFocus((WXHWND)wParam);
-
- case WM_CUT:
- case WM_COPY:
- case WM_PASTE:
- return HandleClipboardEvent(msg);
+ default:
+ if ( ShouldForwardFromEditToCombo(msg) )
+ {
+ // For all the messages forward from the edit control the
+ // result is not used.
+ WXLRESULT result;
+ return MSWHandleMessage(&result, msg, wParam, lParam);
+ }
}
return false;
case CBN_DROPDOWN:
// remember the last selection, just as wxChoice does
m_lastAcceptedSelection = GetCurrentSelection();
- if ( m_lastAcceptedSelection == -1 )
- {
- // but unlike with wxChoice we may have no selection but still
- // have some text and we should avoid erasing it if the drop
- // down is cancelled (see #8474)
- m_lastAcceptedSelection = wxID_NONE;
- }
{
wxCommandEvent event(wxEVT_COMMAND_COMBOBOX_DROPDOWN, GetId());
event.SetEventObject(this);
ProcessCommand(event);
}
break;
+
case CBN_CLOSEUP:
+ // Do the same thing as in wxChoice but using different event type.
+ if ( m_pendingSelection != wxID_NONE )
+ {
+ SendSelectionChangedEvent(wxEVT_COMMAND_COMBOBOX_SELECTED);
+ m_pendingSelection = wxID_NONE;
+ }
{
wxCommandEvent event(wxEVT_COMMAND_COMBOBOX_CLOSEUP, GetId());
event.SetEventObject(this);
ProcessCommand(event);
}
break;
+
case CBN_SELENDOK:
#ifndef __SMARTPHONE__
// we need to reset this to prevent the selection from being undone
// this string is going to become the new combobox value soon but
// we need it to be done right now, otherwise the event handler
// could get a wrong value when it calls our GetValue()
- ::SetWindowText(GetHwnd(), value.wx_str());
+ ::SetWindowText(GetHwnd(), value.t_str());
- {
- wxCommandEvent event(wxEVT_COMMAND_COMBOBOX_SELECTED, GetId());
- event.SetInt(sel);
- event.SetString(value);
- InitCommandEventWithItems(event, sel);
-
- ProcessCommand(event);
- }
+ SendSelectionChangedEvent(wxEVT_COMMAND_COMBOBOX_SELECTED);
- // fall through: for compability with wxGTK, also send the text
+ // fall through: for compatibility with wxGTK, also send the text
// update event when the selection changes (this also seems more
// logical as the text does change)
WXHWND wxComboBox::GetEditHWNDIfAvailable() const
{
-#if wxUSE_DYNLIB_CLASS
+ // FIXME-VC6: Only VC6 needs this guard, see WINVER definition in
+ // include/wx/msw/wrapwin.h
#if defined(WINVER) && WINVER >= 0x0500
- typedef BOOL (WINAPI *GetComboBoxInfo_t)(HWND, COMBOBOXINFO*);
- static GetComboBoxInfo_t s_pfnGetComboBoxInfo = NULL;
- static bool s_triedToLoad = false;
- if ( !s_triedToLoad )
- {
- s_triedToLoad = true;
- wxLoadedDLL dllUser32("user32.dll");
- wxDL_INIT_FUNC(s_pfn, GetComboBoxInfo, dllUser32);
- }
-
- if ( s_pfnGetComboBoxInfo )
- {
- WinStruct<COMBOBOXINFO> info;
- (*s_pfnGetComboBoxInfo)(GetHwnd(), &info);
+ WinStruct<COMBOBOXINFO> info;
+ if ( MSWGetComboBoxInfo(&info) )
return info.hwndItem;
- }
#endif // WINVER >= 0x0500
-#endif // wxUSE_DYNLIB_CLASS
if (HasFlag(wxCB_SIMPLE))
{
wxTextEntry::Clear();
}
+bool wxComboBox::ContainsHWND(WXHWND hWnd) const
+{
+ return hWnd == GetEditHWNDIfAvailable();
+}
+
void wxComboBox::GetSelection(long *from, long *to) const
{
if ( !HasFlag(wxCB_READONLY) )
event.Enable(IsEditable() && !wxTextEntry::IsEmpty());
}
-void wxComboBox::MSWDoPopupOrDismiss(bool show)
-{
- wxASSERT_MSG( !HasFlag(wxCB_SIMPLE),
- wxT("can't popup/dismiss the list for simple combo box") );
-
- // we *must* set focus to the combobox before showing or hiding the drop
- // down as without this we get WM_LBUTTONDOWN messages with invalid HWND
- // when hiding it (whether programmatically or manually) resulting in a
- // crash when we pass them to IsDialogMessage()
- //
- // this can be seen in the combo page of the widgets sample under Windows 7
- SetFocus();
-
- ::SendMessage(GetHwnd(), CB_SHOWDROPDOWN, show, 0);
-}
-
#if wxUSE_TOOLTIPS
void wxComboBox::DoSetToolTip(wxToolTip *tip)
wxChoice::DoSetToolTip(tip);
if ( tip && !HasFlag(wxCB_READONLY) )
- tip->Add(GetEditHWND());
+ tip->AddOtherWindow(GetEditHWND());
}
#endif // wxUSE_TOOLTIPS
#endif // wxUSE_UXTHEME
+wxSize wxComboBox::DoGetSizeFromTextSize(int xlen, int ylen) const
+{
+ wxSize tsize( wxChoice::DoGetSizeFromTextSize(xlen, ylen) );
+
+ if ( !HasFlag(wxCB_READONLY) )
+ {
+ // Add the margins we have previously set
+ wxPoint marg( GetMargins() );
+ marg.x = wxMax(0, marg.x);
+ marg.y = wxMax(0, marg.y);
+ tsize.IncBy( marg );
+ }
+
+ return tsize;
+}
+
#endif // wxUSE_COMBOBOX
projects / wxWidgets.git / blobdiff