support metal appearance

[wxWidgets.git] / src / os2 / combobox.cpp

diff --git a/src/os2/combobox.cpp b/src/os2/combobox.cpp
index bf7feb68607fe3fd64820cffc9dee551fd6e0165..c8e5138c7b4a02517fe88e7bb2986529a4fc1595 100644 (file)
--- a/src/os2/combobox.cpp
+++ b/src/os2/combobox.cpp
@@ -15,7 +15,8 @@
 #include "wx/wxprec.h"
 
 #ifndef WX_PRECOMP
 #include "wx/wxprec.h"
 
 #ifndef WX_PRECOMP

-#include "wx/setup.h"
+    #include "wx/setup.h"
+    #include "wx/settings.h"

 #endif
 
 #if wxUSE_COMBOBOX
 #endif
 
 #if wxUSE_COMBOBOX


@@ -48,7 +49,7 @@ bool wxComboBox::OS2Command(
 
     switch (uParam)
     {
 
     switch (uParam)
     {

-        case LN_SELECT:
+        case CBN_LBSELECT:

             if (GetSelection() > -1)
             {
                 wxCommandEvent      vEvent( wxEVT_COMMAND_COMBOBOX_SELECTED
             if (GetSelection() > -1)
             {
                 wxCommandEvent      vEvent( wxEVT_COMMAND_COMBOBOX_SELECTED


@@ -62,7 +63,7 @@ bool wxComboBox::OS2Command(
             }
             break;
 
             }
             break;
 

-        case EN_CHANGE:
+        case CBN_EFCHANGE:

             {
                 wxCommandEvent      vEvent( wxEVT_COMMAND_TEXT_UPDATED
                                            ,GetId()
             {
                 wxCommandEvent      vEvent( wxEVT_COMMAND_TEXT_UPDATED
                                            ,GetId()


@@ -85,6 +86,24 @@ bool wxComboBox::OS2Command(
     return FALSE;
 } // end of wxComboBox::OS2Command
 
     return FALSE;
 } // end of wxComboBox::OS2Command
 

+bool wxComboBox::Create(
+  wxWindow*                         pParent
+, wxWindowID                        vId
+, const wxString&                   rsValue
+, const wxPoint&                    rPos
+, const wxSize&                     rSize
+, const wxArrayString&              asChoices
+, long                              lStyle
+, const wxValidator&                rValidator
+, const wxString&                   rsName
+)
+{
+    wxCArrayString chs(asChoices);
+
+    return Create(pParent, vId, rsValue, rPos, rSize, chs.GetCount(),
+                  chs.GetStrings(), lStyle, rValidator, rsName);
+}
+

 bool wxComboBox::Create(
   wxWindow*                         pParent
 , wxWindowID                        vId
 bool wxComboBox::Create(
   wxWindow*                         pParent
 , wxWindowID                        vId


@@ -94,21 +113,18 @@ bool wxComboBox::Create(
 , int                               n
 , const wxString                    asChoices[]
 , long                              lStyle
 , int                               n
 , const wxString                    asChoices[]
 , long                              lStyle

-#if wxUSE_VALIDATORS

 , const wxValidator&                rValidator
 , const wxValidator&                rValidator

-#endif

 , const wxString&                   rsName
 )
 {
 , const wxString&                   rsName
 )
 {

+    m_isShown = FALSE;

 
 

-    if (!OS2CreateControl( pParent
+    if (!CreateControl( pParent

                        ,vId
                        ,rPos
                        ,rSize
                        ,lStyle
                        ,vId
                        ,rPos
                        ,rSize
                        ,lStyle

-#if wxUSE_VALIDATORS

                        ,rValidator
                        ,rValidator

-#endif

                        ,rsName
                       ))
         return FALSE;
                        ,rsName
                       ))
         return FALSE;


@@ -140,12 +156,11 @@ bool wxComboBox::Create(
     // A choice/combobox normally has a white background (or other, depending
     // on global settings) rather than inheriting the parent's background colour.
     //
     // A choice/combobox normally has a white background (or other, depending
     // on global settings) rather than inheriting the parent's background colour.
     //

-    SetBackgroundColour(wxSystemSettings::GetSystemColour(wxSYS_COLOUR_WINDOW));
+    SetBackgroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_WINDOW));

 
 

-    SetFont(pParent->GetFont());
+    SetFont(*wxSMALL_FONT);

 
     int                             i;
 
     int                             i;

-

     for (i = 0; i < n; i++)
     {
         Append(asChoices[i]);
     for (i = 0; i < n; i++)
     {
         Append(asChoices[i]);


@@ -164,6 +179,7 @@ bool wxComboBox::Create(
                                                     ,(PFNWP)wxComboEditWndProc
                                                    );
     ::WinSetWindowULong(GetHwnd(), QWL_USER, (ULONG)this);
                                                     ,(PFNWP)wxComboEditWndProc
                                                    );
     ::WinSetWindowULong(GetHwnd(), QWL_USER, (ULONG)this);

+    Show(TRUE);

     return TRUE;
 } // end of wxComboBox::Create
 
     return TRUE;
 } // end of wxComboBox::Create
 


@@ -171,37 +187,8 @@ void wxComboBox::SetValue(
   const wxString&                   rsValue
 )
 {
   const wxString&                   rsValue
 )
 {

-    //
-    // If newlines are denoted by just 10, must stick 13 in front.
-    //
-    int                             nSingletons = 0;
-    int                             nLen = rsValue.Length();
-    int                             i;
-
-    for (i = 0; i < nLen; i ++)
-    {
-        if ((i > 0) && (rsValue[i] == 10) && (rsValue[i - 1] != 13))
-            nSingletons ++;
-    }
-    if (nSingletons > 0)
-    {
-        wxChar*                     zTmp = new wxChar[nLen + nSingletons + 1];
-        int                         j = 0;
-
-        for (i = 0; i < nLen; i ++)
-        {
-            if ((i > 0) && (rsValue[i] == 10) && (rsValue[i - 1] != 13))
-            {
-                zTmp[j] = 13;
-                j++;
-            }
-            zTmp[j] = rsValue[i];
-            j++;
-        }
-        zTmp[j] = 0;
-        ::WinSetWindowText(GetHwnd(), zTmp);
-        delete[] zTmp;
-    }
+    if ( HasFlag(wxCB_READONLY) )
+        SetStringSelection(rsValue);

     else
         ::WinSetWindowText(GetHwnd(), rsValue.c_str());
 } // end of wxComboBox::SetValue
     else
         ::WinSetWindowText(GetHwnd(), rsValue.c_str());
 } // end of wxComboBox::SetValue


@@ -269,7 +256,6 @@ long wxComboBox::GetInsertionPoint() const
 
 long wxComboBox::GetLastPosition() const
 {
 
 long wxComboBox::GetLastPosition() const
 {

-    HWND                            hEditWnd = GetHwnd();

     long                            lLineLength = 0L;
     WNDPARAMS                       vParams;
 
     long                            lLineLength = 0L;
     WNDPARAMS                       vParams;
 


@@ -301,8 +287,6 @@ void wxComboBox::Replace(
 {
 #if wxUSE_CLIPBOARD
     HWND                            hWnd = GetHwnd();
 {
 #if wxUSE_CLIPBOARD
     HWND                            hWnd = GetHwnd();

-    long                            lFromChar = lFrom;
-    long                            lToChar = lTo;

 
     //
     // Set selection and remove it
 
     //
     // Set selection and remove it


@@ -333,8 +317,6 @@ void wxComboBox::Remove(
 {
 #if wxUSE_CLIPBOARD
     HWND                            hWnd = GetHwnd();
 {
 #if wxUSE_CLIPBOARD
     HWND                            hWnd = GetHwnd();

-    long                            lFromChar = lFrom;
-    long                            lToChar = lTo;

 
     ::WinSendMsg(hWnd, EM_SETSEL, MPFROM2SHORT((USHORT)lFrom, (USHORT)lTo), 0);
     ::WinSendMsg(hWnd, EM_CUT, (MPARAM)0, (MPARAM)0);
 
     ::WinSendMsg(hWnd, EM_SETSEL, MPFROM2SHORT((USHORT)lFrom, (USHORT)lTo), 0);
     ::WinSendMsg(hWnd, EM_CUT, (MPARAM)0, (MPARAM)0);


@@ -347,8 +329,8 @@ void wxComboBox::SetSelection(
 )
 {
     HWND                            hWnd = GetHwnd();
 )
 {
     HWND                            hWnd = GetHwnd();

-    long                            lFromChar = lFrom;
-    long                            lToChar = lTo;
+    long                            lFromChar = 0;
+    long                            lToChar   = 0;

 
     //
     // If from and to are both -1, it means
 
     //
     // If from and to are both -1, it means


@@ -397,22 +379,29 @@ bool wxComboBox::ProcessEditMsg(
             switch(vFlag)
             {
                 case KC_CHAR:
             switch(vFlag)
             {
                 case KC_CHAR:

-                    return (HandleChar( SHORT1FROMMP(wParam)
+                    return (HandleChar( wParam

                                        ,lParam
                                        ,TRUE /* isASCII */
                                       ));
 
                 case KC_PREVDOWN:
                                        ,lParam
                                        ,TRUE /* isASCII */
                                       ));
 
                 case KC_PREVDOWN:

-                    return (HandleKeyDown( SHORT1FROMMP(wParam)
+                    return (HandleKeyDown( wParam

                                           ,lParam
                                          ));
 
                 case KC_KEYUP:
                                           ,lParam
                                          ));
 
                 case KC_KEYUP:

-                    return (HandleKeyUp( SHORT1FROMMP(wParam)
+                    return (HandleKeyUp( wParam

                                         ,lParam
                                        ));
             }
             break;
                                         ,lParam
                                        ));
             }
             break;

+
+        case WM_SETFOCUS:
+            if (SHORT1FROMMP((MPARAM)lParam) == TRUE)
+                return(HandleSetFocus((WXHWND)(HWND)wParam));
+            else
+                return(HandleKillFocus((WXHWND)(HWND)wParam));
+            break;

     }
     return FALSE;
 } // end of WinGuiBase_CComboBox::ProcessEditMsg
     }
     return FALSE;
 } // end of WinGuiBase_CComboBox::ProcessEditMsg


@@ -434,6 +423,7 @@ MRESULT EXPENTRY wxComboEditWndProc(
         //
         // Forward some messages to the combobox
         //
         //
         // Forward some messages to the combobox
         //

+        case WM_SETFOCUS:

         case WM_CHAR:
             {
                 wxComboBox*         pCombo = wxDynamicCast( pWin
         case WM_CHAR:
             {
                 wxComboBox*         pCombo = wxDynamicCast( pWin