Typo fix

[wxWidgets.git] / src / msw / toplevel.cpp

diff --git a/src/msw/toplevel.cpp b/src/msw/toplevel.cpp
index e77677c7d40645d6afe677aee0f383780a00a0a3..8283ec20d63680fad47c24cc543ed5a1ac2c2982 100644 (file)
--- a/src/msw/toplevel.cpp
+++ b/src/msw/toplevel.cpp
@@ -39,6 +39,14 @@
 
 #include "wx/msw/private.h"
 
 
 #include "wx/msw/private.h"
 

+#ifndef ICON_BIG
+    #define ICON_BIG 1
+#endif
+
+#ifndef ICON_SMALL
+    #define ICON_SMALL 0
+#endif
+

 // ----------------------------------------------------------------------------
 // stubs for missing functions under MicroWindows
 // ----------------------------------------------------------------------------
 // ----------------------------------------------------------------------------
 // stubs for missing functions under MicroWindows
 // ----------------------------------------------------------------------------


@@ -64,21 +72,25 @@ extern const wxChar *wxCanvasClassName;
 // wxTopLevelWindowMSW implementation
 // ============================================================================
 
 // wxTopLevelWindowMSW implementation
 // ============================================================================
 

+// ----------------------------------------------------------------------------
+// wxDialog helpers
+// ----------------------------------------------------------------------------
+

 // Dialog window proc
 LONG APIENTRY _EXPORT
 // Dialog window proc
 LONG APIENTRY _EXPORT

-wxDlgProc(HWND WXUNUSED(hWnd), UINT message, WPARAM WXUNUSED(wParam), LPARAM WXUNUSED(lParam))
+wxDlgProc(HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam)

 {
 {

-    if ( message == WM_INITDIALOG )
+    switch ( message )

     {
     {

-        // for this message, returning TRUE tells system to set focus to the
-        // first control in the dialog box
-        return TRUE;
-    }
-    else
-    {
-        // for all the other ones, FALSE means that we didn't process the
-        // message
-        return FALSE;
+        case WM_INITDIALOG:
+            // for this message, returning TRUE tells system to set focus to the
+            // first control in the dialog box
+            return TRUE;
+
+        default:
+            // for all the other ones, FALSE means that we didn't process the
+            // message
+            return FALSE;

     }
 }
 
     }
 }
 


@@ -101,20 +113,24 @@ void wxTopLevelWindowMSW::Init()
     m_fsIsShowing = FALSE;
 }
 
     m_fsIsShowing = FALSE;
 }
 

-long wxTopLevelWindowMSW::MSWGetCreateWindowFlags(long *exflags) const
+WXDWORD wxTopLevelWindowMSW::MSWGetStyle(long style, WXDWORD *exflags) const

 {
 {

-    long style = GetWindowStyle();
+    // let the base class deal with the common styles but fix the ones which
+    // don't make sense for us (we also deal with the borders ourselves)
+    WXDWORD msflags = wxWindow::MSWGetStyle
+                      (
+                        (style & ~wxBORDER_MASK) | wxBORDER_NONE, exflags
+                      ) & ~WS_CHILD;

 
     // first select the kind of window being created
     //
     // note that if we don't set WS_POPUP, Windows assumes WS_OVERLAPPED and
     // creates a window with both caption and border, hence we also test it
     // below in some other cases
 
     // first select the kind of window being created
     //
     // note that if we don't set WS_POPUP, Windows assumes WS_OVERLAPPED and
     // creates a window with both caption and border, hence we also test it
     // below in some other cases

-    long msflags;

     if ( style & wxFRAME_TOOL_WINDOW )
     if ( style & wxFRAME_TOOL_WINDOW )

-        msflags = WS_POPUP;
+        msflags |= WS_POPUP;

     else
     else

-        msflags = WS_OVERLAPPED;
+        msflags |= WS_OVERLAPPED;

 
     // border and caption styles
     if ( style & wxRESIZE_BORDER )
 
     // border and caption styles
     if ( style & wxRESIZE_BORDER )


@@ -141,9 +157,6 @@ long wxTopLevelWindowMSW::MSWGetCreateWindowFlags(long *exflags) const
     if ( style & wxMAXIMIZE )
         msflags |= WS_MAXIMIZE;
 
     if ( style & wxMAXIMIZE )
         msflags |= WS_MAXIMIZE;
 

-    if ( style & wxCLIP_CHILDREN )
-        msflags |= WS_CLIPCHILDREN;
-

     // Keep this here because it saves recoding this function in wxTinyFrame
 #if wxUSE_ITSY_BITSY && !defined(__WIN32__)
     if ( style & wxTINY_CAPTION_VERT )
     // Keep this here because it saves recoding this function in wxTinyFrame
 #if wxUSE_ITSY_BITSY && !defined(__WIN32__)
     if ( style & wxTINY_CAPTION_VERT )


@@ -157,8 +170,6 @@ long wxTopLevelWindowMSW::MSWGetCreateWindowFlags(long *exflags) const
 
     if ( exflags )
     {
 
     if ( exflags )
     {

-        *exflags = MakeExtendedStyle(style);
-

 #if !defined(__WIN16__) && !defined(__SC__)
         if ( !(GetExtraStyle() & wxTOPLEVEL_EX_DIALOG) )
         {
 #if !defined(__WIN16__) && !defined(__SC__)
         if ( !(GetExtraStyle() & wxTOPLEVEL_EX_DIALOG) )
         {


@@ -177,7 +188,7 @@ long wxTopLevelWindowMSW::MSWGetCreateWindowFlags(long *exflags) const
             *exflags |= WS_EX_TOPMOST;
 
 #ifdef __WIN32__
             *exflags |= WS_EX_TOPMOST;
 
 #ifdef __WIN32__

-        if ( m_exStyle & wxFRAME_EX_CONTEXTHELP )
+        if ( GetExtraStyle() & wxFRAME_EX_CONTEXTHELP )

             *exflags |= WS_EX_CONTEXTHELP;
 #endif // __WIN32__
     }
             *exflags |= WS_EX_CONTEXTHELP;
 #endif // __WIN32__
     }


@@ -218,10 +229,13 @@ bool wxTopLevelWindowMSW::CreateDialog(const void *dlgTemplate,
         }
     }
 
         }
     }
 

-    m_hWnd = (WXHWND)::CreateDialogIndirect(wxGetInstance(),
-                                    (DLGTEMPLATE*)dlgTemplate,
-                                    parent ? GetHwndOf(parent) : NULL,
-                                    (DLGPROC)wxDlgProc);
+    m_hWnd = (WXHWND)::CreateDialogIndirect
+                       (
+                        wxGetInstance(),
+                        (DLGTEMPLATE*)dlgTemplate,
+                        parent ? GetHwndOf(parent) : NULL,
+                        (DLGPROC)wxDlgProc
+                       );

 
     if ( !m_hWnd )
     {
 
     if ( !m_hWnd )
     {


@@ -232,7 +246,7 @@ bool wxTopLevelWindowMSW::CreateDialog(const void *dlgTemplate,
         return FALSE;
     }
 
         return FALSE;
     }
 

-    long exflags;
+    WXDWORD exflags;

     (void)MSWGetCreateWindowFlags(&exflags);
 
     if ( exflags )
     (void)MSWGetCreateWindowFlags(&exflags);
 
     if ( exflags )


@@ -315,8 +329,8 @@ bool wxTopLevelWindowMSW::CreateFrame(const wxString& title,
                                       const wxPoint& pos,
                                       const wxSize& size)
 {
                                       const wxPoint& pos,
                                       const wxSize& size)
 {

-    long exflags;
-    long flags = MSWGetCreateWindowFlags(&exflags);
+    WXDWORD exflags;
+    WXDWORD flags = MSWGetCreateWindowFlags(&exflags);

 
     return MSWCreate(wxCanvasClassName, title, pos, size, flags, exflags);
 }
 
     return MSWCreate(wxCanvasClassName, title, pos, size, flags, exflags);
 }


@@ -345,36 +359,37 @@ bool wxTopLevelWindowMSW::Create(wxWindow *parent,
 
     if ( GetExtraStyle() & wxTOPLEVEL_EX_DIALOG )
     {
 
     if ( GetExtraStyle() & wxTOPLEVEL_EX_DIALOG )
     {

-        // TODO: it would be better to construct the dialog template in memory
-        //       during run-time than to rely on the limited number of
-        //       templates in wx.rc because:
-        //          a) you wouldn't have to include wx.rc in all wxWin programs
-        //             (and the number of complaints about it would dtop)
-        //          b) we'd be able to provide more templates simply, i.e.
-        //             we could generate the templates for all style
-        //             combinations
-

         // we have different dialog templates to allows creation of dialogs
         // with & without captions under MSWindows, resizeable or not (but a
         // resizeable dialog always has caption - otherwise it would look too
         // strange)
         // we have different dialog templates to allows creation of dialogs
         // with & without captions under MSWindows, resizeable or not (but a
         // resizeable dialog always has caption - otherwise it would look too
         // strange)

-        int dlgsize = sizeof(DLGTEMPLATE) + (sizeof(WORD) * 3);
-        DLGTEMPLATE* dlgTemplate = (DLGTEMPLATE*)malloc( dlgsize );
-        memset (dlgTemplate, 0, dlgsize );
+
+        // we need 3 additional WORDs for dialog menu, class and title (as we
+        // don't use DS_SETFONT we don't need the fourth WORD for the font)
+        static const int dlgsize = sizeof(DLGTEMPLATE) + (sizeof(WORD) * 3);
+        DLGTEMPLATE *dlgTemplate = (DLGTEMPLATE *)malloc(dlgsize);
+        memset(dlgTemplate, 0, dlgsize);
+
+        // these values are arbitrary, they won't be used normally anyhow

         dlgTemplate->x  = 34;
         dlgTemplate->y  = 22;
         dlgTemplate->cx = 144;
         dlgTemplate->cy = 75;
 
         dlgTemplate->x  = 34;
         dlgTemplate->y  = 22;
         dlgTemplate->cx = 144;
         dlgTemplate->cy = 75;
 

-        if ( style & wxRESIZE_BORDER )
-          dlgTemplate->style = DS_MODALFRAME | WS_CAPTION | WS_POPUP | WS_SYSMENU | WS_THICKFRAME;
-        else if ( style & wxCAPTION )
-          dlgTemplate->style = DS_MODALFRAME | WS_CAPTION | WS_POPUP | WS_SYSMENU;
-        else
-          dlgTemplate->style = WS_POPUP;
+        // reuse the code in MSWGetStyle() but correct the results slightly for
+        // the dialog
+        dlgTemplate->style = MSWGetStyle(style, NULL);
+
+        // all dialogs are popups
+        dlgTemplate->style |= WS_POPUP;
+
+        // force 3D-look if necessary, it looks impossibly ugly otherwise
+        if ( style & (wxRESIZE_BORDER | wxCAPTION) )
+            dlgTemplate->style |= DS_MODALFRAME;

 
         bool ret = CreateDialog(dlgTemplate, title, pos, size);
         free(dlgTemplate);
 
         bool ret = CreateDialog(dlgTemplate, title, pos, size);
         free(dlgTemplate);

+

         return ret;
     }
     else // !dialog
         return ret;
     }
     else // !dialog


@@ -581,14 +596,26 @@ bool wxTopLevelWindowMSW::ShowFullScreen(bool show, long style)
 
 void wxTopLevelWindowMSW::SetIcon(const wxIcon& icon)
 {
 
 void wxTopLevelWindowMSW::SetIcon(const wxIcon& icon)
 {

-    // this sets m_icon
-    wxTopLevelWindowBase::SetIcon(icon);
+    SetIcons( wxIconBundle( icon ) );
+}
+
+void wxTopLevelWindowMSW::SetIcons(const wxIconBundle& icons)
+{
+    wxTopLevelWindowBase::SetIcons(icons);

 
 #if defined(__WIN95__) && !defined(__WXMICROWIN__)
 
 #if defined(__WIN95__) && !defined(__WXMICROWIN__)

-    if ( m_icon.Ok() )
+    const wxIcon& sml = icons.GetIcon( wxSize( 16, 16 ) );
+    if( sml.Ok() && sml.GetWidth() == 16 && sml.GetHeight() == 16 )
+    {
+        ::SendMessage( GetHwndOf( this ), WM_SETICON, ICON_SMALL,
+                       (LPARAM)GetHiconOf(sml) );
+    }
+
+    const wxIcon& big = icons.GetIcon( wxSize( 32, 32 ) );
+    if( big.Ok() && big.GetWidth() == 32 && big.GetHeight() == 32 )

     {
     {

-        ::SendMessage(GetHwnd(), WM_SETICON,
-                      (WPARAM)TRUE, (LPARAM)GetHiconOf(m_icon));
+        ::SendMessage( GetHwndOf( this ), WM_SETICON, ICON_BIG,
+                       (LPARAM)GetHiconOf(big) );

     }
 #endif // __WIN95__
 }
     }
 #endif // __WIN95__
 }