X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/0e320a79f187558effb04d92020b470372bbe456..8d813b20f78c6e24d7e0f3b8dbe8de4cb3689db4:/src/os2/choice.cpp diff --git a/src/os2/choice.cpp b/src/os2/choice.cpp index a584d5303f..84758c1bb8 100644 --- a/src/os2/choice.cpp +++ b/src/os2/choice.cpp @@ -1,119 +1,288 @@ ///////////////////////////////////////////////////////////////////////////// // Name: choice.cpp // Purpose: wxChoice -// Author: AUTHOR +// Author: David Webster // Modified by: -// Created: ??/??/98 +// Created: 10/13/99 // RCS-ID: $Id$ -// Copyright: (c) AUTHOR -// Licence: wxWindows licence +// Copyright: (c) David Webster +// Licence: wxWindows licence ///////////////////////////////////////////////////////////////////////////// -#ifdef __GNUG__ -#pragma implementation "choice.h" +// For compilers that support precompilation, includes "wx.h". +#include "wx/wxprec.h" + +#ifndef WX_PRECOMP + #include "wx/choice.h" + #include "wx/utils.h" + #include "wx/log.h" #endif -// For compilers that support precompilation, includes "wx.h". -#include "wx/choice.h" +#include "wx/os2/private.h" -#if !USE_SHARED_LIBRARY IMPLEMENT_DYNAMIC_CLASS(wxChoice, wxControl) -#endif -bool wxChoice::Create(wxWindow *parent, wxWindowID id, - const wxPoint& pos, - const wxSize& size, - int n, const wxString choices[], - long style, - const wxValidator& validator, - const wxString& name) +bool wxChoice::Create(wxWindow *parent, + wxWindowID id, + const wxPoint& pos, + const wxSize& size, + int n, const wxString choices[], + long style, +#if wxUSE_VALIDATORS + const wxValidator& validator, +#endif + const wxString& name) { - SetName(name); - SetValidator(validator); - m_noStrings = n; - m_windowStyle = style; + if ( !CreateControl(parent, id, pos, size, style, validator, name) ) + return FALSE; +// TODO: +/* + long msStyle = WS_CHILD | CBS_DROPDOWNLIST | WS_TABSTOP | WS_VISIBLE | WS_HSCROLL | WS_VSCROLL; + if ( style & wxCB_SORT ) + msStyle |= CBS_SORT; - if (parent) parent->AddChild(this); + // the experience shows that wxChoice vs. wxComboBox distinction confuses + // quite a few people - try to help them + wxASSERT_MSG( !(style & wxCB_DROPDOWN) && + !(style & wxCB_READONLY) && + !(style & wxCB_SIMPLE), + wxT("this style flag is ignored by wxChoice, you " + "probably want to use a wxComboBox") ); - if ( id == -1 ) - m_windowId = (int)NewControlId(); - else - m_windowId = id; + if ( !OS2CreateControl(wxT("COMBOBOX"), msStyle) ) + return FALSE; - // TODO: create choice control - return FALSE; + for ( int i = 0; i < n; i++ ) + { + Append(choices[i]); + } +*/ + SetSize(pos.x, pos.y, size.x, size.y); + + return TRUE; } -void wxChoice::Append(const wxString& item) +// ---------------------------------------------------------------------------- +// adding/deleting items to/from the list +// ---------------------------------------------------------------------------- + +int wxChoice::DoAppend(const wxString& item) { - // TODO - m_noStrings ++; + // TODO: + /* + int n = (int)SendMessage(GetHwnd(), CB_ADDSTRING, 0, (LONG)item.c_str()); + if ( n == CB_ERR ) + { + wxLogLastError("SendMessage(CB_ADDSTRING)"); + } + */ + return 0; //n } void wxChoice::Delete(int n) { - // TODO - m_noStrings --; + wxCHECK_RET( n < GetCount(), wxT("invalid item index in wxChoice::Delete") ); + +// TODO: SendMessage(GetHwnd(), CB_DELETESTRING, n, 0); } void wxChoice::Clear() { - // TODO - m_noStrings = 0; + // TODO: SendMessage(GetHwnd(), CB_RESETCONTENT, 0, 0); } +// ---------------------------------------------------------------------------- +// selection +// ---------------------------------------------------------------------------- + int wxChoice::GetSelection() const { - // TODO + // TODO: return (int)SendMessage(GetHwnd(), CB_GETCURSEL, 0, 0); return 0; } void wxChoice::SetSelection(int n) { - // TODO + // TODO: SendMessage(GetHwnd(), CB_SETCURSEL, n, 0); +} + +// ---------------------------------------------------------------------------- +// string list functions +// ---------------------------------------------------------------------------- + +int wxChoice::GetCount() const +{ + // TODO: return (int)SendMessage(GetHwnd(), CB_GETCOUNT, 0, 0); + return 0; } int wxChoice::FindString(const wxString& s) const { - // TODO + // TODO: + /* + int pos = (int)SendMessage(GetHwnd(), CB_FINDSTRINGEXACT, + (WPARAM)-1, (LPARAM)s.c_str()); + + return pos == LB_ERR ? wxNOT_FOUND : pos; + */ return 0; } +void wxChoice::SetString(int n, const wxString& s) +{ + wxFAIL_MSG(wxT("not implemented")); + +#if 0 // should do this, but no Insert() so far + Delete(n); + Insert(n + 1, s); +#endif +} + wxString wxChoice::GetString(int n) const { - // TODO - return wxString(""); + size_t len = 0; // TODO: (size_t)::SendMessage(GetHwnd(), CB_GETLBTEXTLEN, n, 0); + wxString str = ""; + // TODO: + /* + if (len) { + if ( ::SendMessage(GetHwnd(), CB_GETLBTEXT, n, + (LPARAM)str.GetWriteBuf(len)) == CB_ERR ) { + wxLogLastError("SendMessage(CB_GETLBTEXT)"); + } + str.UngetWriteBuf(); + } + */ + return str; } -void wxChoice::SetSize(int x, int y, int width, int height, int sizeFlags) +// ---------------------------------------------------------------------------- +// client data +// ---------------------------------------------------------------------------- + +void wxChoice::DoSetItemClientData( int n, void* clientData ) { - // TODO + // TODO: + /* + if ( SendMessage(GetHwnd(), CB_SETITEMDATA, n, (LPARAM)clientData) == CB_ERR ) + { + wxLogLastError(wxT("CB_SETITEMDATA")); + } + */ } -wxString wxChoice::GetStringSelection () const +void* wxChoice::DoGetItemClientData( int n ) const { - int sel = GetSelection (); - if (sel > -1) - return wxString(this->GetString (sel)); - else - return wxString(""); + // TODO: + /* + LPARAM rc = SendMessage(GetHwnd(), CB_GETITEMDATA, n, 0); + if ( rc == CB_ERR ) + { + wxLogLastError(wxT("CB_GETITEMDATA")); + + // unfortunately, there is no way to return an error code to the user + rc = (LPARAM) NULL; + } + + return (void *)rc; + */ + return NULL; } -bool wxChoice::SetStringSelection (const wxString& s) +void wxChoice::DoSetItemClientObject( int n, wxClientData* clientData ) { - int sel = FindString (s); - if (sel > -1) - { - SetSelection (sel); - return TRUE; - } - else - return FALSE; + DoSetItemClientData(n, clientData); +} + +wxClientData* wxChoice::DoGetItemClientObject( int n ) const +{ + return (wxClientData *)DoGetItemClientData(n); +} + +// ---------------------------------------------------------------------------- +// wxOS2 specific helpers +// ---------------------------------------------------------------------------- + +void wxChoice::DoSetSize(int x, int y, + int width, int height, + int sizeFlags) +{ + // Ignore height parameter because height doesn't mean 'initially + // displayed' height, it refers to the drop-down menu as well. The + // wxWindows interpretation is different; also, getting the size returns + // the _displayed_ size (NOT the drop down menu size) so + // setting-getting-setting size would not work. + wxControl::DoSetSize(x, y, width, -1, sizeFlags); +} + +wxSize wxChoice::DoGetBestSize() const +{ + // find the widest string + int wLine; + int wChoice = 0; + int nItems = GetCount(); + for ( int i = 0; i < nItems; i++ ) + { + wxString str(GetString(i)); + GetTextExtent(str, &wLine, NULL); + if ( wLine > wChoice ) + wChoice = wLine; + } + + // give it some reasonable default value if there are no strings in the + // list + if ( wChoice == 0 ) + wChoice = 100; + + // the combobox should be larger than the widest string + int cx, cy; + wxGetCharSize(GetHWND(), &cx, &cy, (wxFont*)&GetFont()); + + wChoice += 5*cx; + + // Choice drop-down list depends on number of items (limited to 10) + size_t nStrings = nItems == 0 ? 10 : wxMin(10, nItems) + 1; + int hChoice = EDIT_HEIGHT_FROM_CHAR_HEIGHT(cy)*nStrings; + + return wxSize(wChoice, hChoice); } -void wxChoice::Command(wxCommandEvent & event) +MRESULT wxChoice::OS2WindowProc(WXUINT nMsg, WXWPARAM wParam, WXLPARAM lParam) { - SetSelection (event.GetInt()); - ProcessCommand (event); + // TODO: + /* + if ( nMsg == WM_LBUTTONUP ) + { + int x = (int)LOWORD(lParam); + int y = (int)HIWORD(lParam); + + // Ok, this is truly weird, but if a panel with a wxChoice loses the + // focus, then you get a *fake* WM_LBUTTONUP message with x = 65535 and + // y = 65535. Filter out this nonsense. + // + // VZ: I'd like to know how to reproduce this please... + if ( x == 65535 && y == 65535 ) + return 0; + } + */ + return wxWindow::OS2WindowProc(nMsg, wParam, lParam); +} + +bool wxChoice::OS2Command(WXUINT param, WXWORD WXUNUSED(id)) +{ + // TODO: + /* + if ( param != CBN_SELCHANGE) + { + // "selection changed" is the only event we're after + return FALSE; + } + */ + wxCommandEvent event(wxEVT_COMMAND_CHOICE_SELECTED, m_windowId); + event.SetInt(GetSelection()); + event.SetEventObject(this); +// TODO: event.SetString(GetStringSelection()); + ProcessCommand(event); + + return TRUE; }