Put some life into GTK 2.0 drawing.

[wxWidgets.git] / src / univ / button.cpp

diff --git a/src/univ/button.cpp b/src/univ/button.cpp
index 7a7a9fa771bda7f31ce86b69adc435103b8f45f7..eaf7fea91bd05ef5d1f7256f7fe60f4419d41060 100644 (file)
--- a/src/univ/button.cpp
+++ b/src/univ/button.cpp
@@ -18,7 +18,7 @@
 // ----------------------------------------------------------------------------
 
 #ifdef __GNUG__
 // ----------------------------------------------------------------------------
 
 #ifdef __GNUG__

-    #pragma implementation "button.h"
+    #pragma implementation "univbutton.h"

 #endif
 
 #include "wx/wxprec.h"
 #endif
 
 #include "wx/wxprec.h"


@@ -45,7 +45,7 @@
 // ----------------------------------------------------------------------------
 
 // default margins around the image
 // ----------------------------------------------------------------------------
 
 // default margins around the image

-static const wxCoord DEFAULT_BTN_MARGIN_X = 0;
+static const wxCoord DEFAULT_BTN_MARGIN_X = 0;  // We should give space for the border, at least.

 static const wxCoord DEFAULT_BTN_MARGIN_Y = 0;
 
 // ============================================================================
 static const wxCoord DEFAULT_BTN_MARGIN_Y = 0;
 
 // ============================================================================


@@ -136,9 +136,12 @@ wxSize wxButton::DoGetBestClientSize() const
 
     // for compatibility with other ports, the buttons default size is never
     // less than the standard one
 
     // for compatibility with other ports, the buttons default size is never
     // less than the standard one

-    wxSize szDef = GetDefaultSize();
-    if ( width < szDef.x )
-        width = szDef.x;
+    if ( !(GetWindowStyle() & wxBU_EXACTFIT) )
+    {
+        wxSize szDef = GetDefaultSize();
+        if ( width < szDef.x )
+            width = szDef.x;
+    }

 
     return wxSize(width, height);
 }
 
     return wxSize(width, height);
 }


@@ -149,7 +152,11 @@ wxSize wxButton::DoGetBestClientSize() const
 
 void wxButton::DoDraw(wxControlRenderer *renderer)
 {
 
 void wxButton::DoDraw(wxControlRenderer *renderer)
 {

-    renderer->DrawButtonBorder();
+    if ( !(GetWindowStyle() & wxBORDER_NONE) )
+    {
+        renderer->DrawButtonBorder();
+    }
+

     renderer->DrawLabel(m_bitmap, m_marginBmpX, m_marginBmpY);
 }
 
     renderer->DrawLabel(m_bitmap, m_marginBmpX, m_marginBmpY);
 }
 


@@ -251,22 +258,22 @@ wxStdButtonInputHandler::wxStdButtonInputHandler(wxInputHandler *handler)
     m_winHasMouse = FALSE;
 }
 
     m_winHasMouse = FALSE;
 }
 

-bool wxStdButtonInputHandler::HandleKey(wxControl *control,
+bool wxStdButtonInputHandler::HandleKey(wxInputConsumer *consumer,

                                         const wxKeyEvent& event,
                                         bool pressed)
 {
     int keycode = event.GetKeyCode();
     if ( keycode == WXK_SPACE || keycode == WXK_RETURN )
     {
                                         const wxKeyEvent& event,
                                         bool pressed)
 {
     int keycode = event.GetKeyCode();
     if ( keycode == WXK_SPACE || keycode == WXK_RETURN )
     {

-        control->PerformAction(wxACTION_BUTTON_TOGGLE);
+        consumer->PerformAction(wxACTION_BUTTON_TOGGLE);

 
         return TRUE;
     }
 
 
         return TRUE;
     }
 

-    return wxStdInputHandler::HandleKey(control, event, pressed);
+    return wxStdInputHandler::HandleKey(consumer, event, pressed);

 }
 
 }
 

-bool wxStdButtonInputHandler::HandleMouse(wxControl *control,
+bool wxStdButtonInputHandler::HandleMouse(wxInputConsumer *consumer,

                                           const wxMouseEvent& event)
 {
     // the button has 2 states: pressed and normal with the following
                                           const wxMouseEvent& event)
 {
     // the button has 2 states: pressed and normal with the following


@@ -279,17 +286,17 @@ bool wxStdButtonInputHandler::HandleMouse(wxControl *control,
     // the other mouse buttons are ignored
     if ( event.Button(1) )
     {
     // the other mouse buttons are ignored
     if ( event.Button(1) )
     {

-        if ( event.ButtonDown(1) )
+        if ( event.LeftDown() || event.LeftDClick() )

         {
         {

-            m_winCapture = control;
+            m_winCapture = consumer->GetInputWindow();

             m_winCapture->CaptureMouse();
             m_winHasMouse = TRUE;
 
             m_winCapture->CaptureMouse();
             m_winHasMouse = TRUE;
 

-            control->PerformAction(wxACTION_BUTTON_PRESS);
+            consumer->PerformAction(wxACTION_BUTTON_PRESS);

 
             return TRUE;
         }
 
             return TRUE;
         }

-        else // up
+        else if ( event.LeftUp() )

         {
             if ( m_winCapture )
             {
         {
             if ( m_winCapture )
             {


@@ -300,19 +307,20 @@ bool wxStdButtonInputHandler::HandleMouse(wxControl *control,
             if ( m_winHasMouse )
             {
                 // this will generate a click event
             if ( m_winHasMouse )
             {
                 // this will generate a click event

-                control->PerformAction(wxACTION_BUTTON_TOGGLE);
+                consumer->PerformAction(wxACTION_BUTTON_TOGGLE);

 
                 return TRUE;
             }
             //else: the mouse was released outside the window, this doesn't
             //      count as a click
         }
 
                 return TRUE;
             }
             //else: the mouse was released outside the window, this doesn't
             //      count as a click
         }

+        //else: don't do anything special about the double click

     }
 
     }
 

-    return wxStdInputHandler::HandleMouse(control, event);
+    return wxStdInputHandler::HandleMouse(consumer, event);

 }
 
 }
 

-bool wxStdButtonInputHandler::HandleMouseMove(wxControl *control,
+bool wxStdButtonInputHandler::HandleMouseMove(wxInputConsumer *consumer,

                                               const wxMouseEvent& event)
 {
     // we only have to do something when the mouse leaves/enters the pressed
                                               const wxMouseEvent& event)
 {
     // we only have to do something when the mouse leaves/enters the pressed


@@ -326,8 +334,8 @@ bool wxStdButtonInputHandler::HandleMouseMove(wxControl *control,
             m_winHasMouse = FALSE;
 
             // we do have a pressed button, so release it
             m_winHasMouse = FALSE;
 
             // we do have a pressed button, so release it

-            control->SetCurrent(FALSE);
-            control->PerformAction(wxACTION_BUTTON_RELEASE);
+            consumer->GetInputWindow()->SetCurrent(FALSE);
+            consumer->PerformAction(wxACTION_BUTTON_RELEASE);

 
             return TRUE;
         }
 
             return TRUE;
         }


@@ -340,17 +348,17 @@ bool wxStdButtonInputHandler::HandleMouseMove(wxControl *control,
 
             // we did have a pressed button which we released when leaving the
             // window, press it again
 
             // we did have a pressed button which we released when leaving the
             // window, press it again

-            control->SetCurrent(TRUE);
-            control->PerformAction(wxACTION_BUTTON_PRESS);
+            consumer->GetInputWindow()->SetCurrent(TRUE);
+            consumer->PerformAction(wxACTION_BUTTON_PRESS);

 
             return TRUE;
         }
     }
 
 
             return TRUE;
         }
     }
 

-    return wxStdInputHandler::HandleMouseMove(control, event);
+    return wxStdInputHandler::HandleMouseMove(consumer, event);

 }
 
 }
 

-bool wxStdButtonInputHandler::HandleFocus(wxControl *control,
+bool wxStdButtonInputHandler::HandleFocus(wxInputConsumer *consumer,

                                           const wxFocusEvent& event)
 {
     // buttons change appearance when they get/lose focus, so return TRUE to
                                           const wxFocusEvent& event)
 {
     // buttons change appearance when they get/lose focus, so return TRUE to


@@ -358,12 +366,12 @@ bool wxStdButtonInputHandler::HandleFocus(wxControl *control,
     return TRUE;
 }
 
     return TRUE;
 }
 

-bool wxStdButtonInputHandler::HandleActivation(wxControl *control,
+bool wxStdButtonInputHandler::HandleActivation(wxInputConsumer *consumer,

                                                bool activated)
 {
     // the default button changes appearance when the app is [de]activated, so
     // return TRUE to refresh
                                                bool activated)
 {
     // the default button changes appearance when the app is [de]activated, so
     // return TRUE to refresh

-    return wxStaticCast(control, wxButton)->IsDefault();
+    return wxStaticCast(consumer->GetInputWindow(), wxButton)->IsDefault();

 }
 
 #endif // wxUSE_BUTTON
 }
 
 #endif // wxUSE_BUTTON