]> git.saurik.com Git - wxWidgets.git/blobdiff - src/generic/calctrl.cpp
fix for toolbar radio buttons event generation
[wxWidgets.git] / src / generic / calctrl.cpp
index dca2a4e76da8f3c22c3619f55f79f48863a147f4..14c60e526734892fd0fdeb3f432daa230ba11a30 100644 (file)
@@ -36,6 +36,8 @@
     #include "wx/stattext.h"
 #endif //WX_PRECOMP
 
+#if wxUSE_CALENDARCTRL
+
 #include "wx/calctrl.h"
 
 #define DEBUG_PAINT 0
@@ -81,9 +83,6 @@ BEGIN_EVENT_TABLE(wxCalendarCtrl, wxControl)
 
     EVT_LEFT_DOWN(wxCalendarCtrl::OnClick)
     EVT_LEFT_DCLICK(wxCalendarCtrl::OnDClick)
-
-    EVT_CALENDAR_MONTH(-1, wxCalendarCtrl::OnCalMonthChange)
-    EVT_CALENDAR_YEAR(-1, wxCalendarCtrl::OnCalMonthChange)
 END_EVENT_TABLE()
 
 BEGIN_EVENT_TABLE(wxMonthComboBox, wxComboBox)
@@ -95,6 +94,18 @@ BEGIN_EVENT_TABLE(wxYearSpinCtrl, wxSpinCtrl)
 END_EVENT_TABLE()
 
 IMPLEMENT_DYNAMIC_CLASS(wxCalendarCtrl, wxControl)
+IMPLEMENT_DYNAMIC_CLASS(wxCalendarEvent, wxCommandEvent)
+
+// ----------------------------------------------------------------------------
+// events
+// ----------------------------------------------------------------------------
+
+DEFINE_EVENT_TYPE(wxEVT_CALENDAR_SEL_CHANGED)
+DEFINE_EVENT_TYPE(wxEVT_CALENDAR_DAY_CHANGED)
+DEFINE_EVENT_TYPE(wxEVT_CALENDAR_MONTH_CHANGED)
+DEFINE_EVENT_TYPE(wxEVT_CALENDAR_YEAR_CHANGED)
+DEFINE_EVENT_TYPE(wxEVT_CALENDAR_DOUBLECLICKED)
+DEFINE_EVENT_TYPE(wxEVT_CALENDAR_WEEKDAY_CLICKED)
 
 // ============================================================================
 // implementation
@@ -110,7 +121,7 @@ wxMonthComboBox::wxMonthComboBox(wxCalendarCtrl *cal)
                             wxDefaultPosition,
                             wxDefaultSize,
                             0, NULL,
-                            wxCB_READONLY)
+                            wxCB_READONLY | wxCLIP_SIBLINGS)
 {
     m_cal = cal;
 
@@ -121,6 +132,7 @@ wxMonthComboBox::wxMonthComboBox(wxCalendarCtrl *cal)
     }
 
     SetSelection(m_cal->GetDate().GetMonth());
+    SetSize(-1, -1, -1, -1, wxSIZE_AUTO_WIDTH|wxSIZE_AUTO_HEIGHT);
 }
 
 wxYearSpinCtrl::wxYearSpinCtrl(wxCalendarCtrl *cal)
@@ -128,7 +140,7 @@ wxYearSpinCtrl::wxYearSpinCtrl(wxCalendarCtrl *cal)
                            cal->GetDate().Format(_T("%Y")),
                            wxDefaultPosition,
                            wxDefaultSize,
-                           wxSP_ARROW_KEYS,
+                           wxSP_ARROW_KEYS | wxCLIP_SIBLINGS,
                            -4300, 10000, cal->GetDate().GetYear())
 {
     m_cal = cal;
@@ -157,9 +169,8 @@ void wxCalendarCtrl::Init()
         m_attrs[n] = NULL;
     }
 
-    wxSystemSettings ss;
-    m_colHighlightFg = ss.GetSystemColour(wxSYS_COLOUR_HIGHLIGHTTEXT);
-    m_colHighlightBg = ss.GetSystemColour(wxSYS_COLOUR_HIGHLIGHT);
+    m_colHighlightFg = wxSystemSettings::GetColour(wxSYS_COLOUR_HIGHLIGHTTEXT);
+    m_colHighlightBg = wxSystemSettings::GetColour(wxSYS_COLOUR_HIGHLIGHT);
 
     m_colHolidayFg = *wxRED;
     // don't set m_colHolidayBg - by default, same as our bg colour
@@ -168,28 +179,41 @@ void wxCalendarCtrl::Init()
     m_colHeaderBg = *wxLIGHT_GREY;
 }
 
-bool wxCalendarCtrl::Create(wxWindow * WXUNUSED(parent),
-                            wxWindowID WXUNUSED(id),
+bool wxCalendarCtrl::Create(wxWindow *parent,
+                            wxWindowID id,
                             const wxDateTime& date,
-                            const wxPoint& WXUNUSED(pos),
+                            const wxPoint& pos,
                             const wxSize& size,
                             long style,
-                            const wxString& WXUNUSED(name))
+                            const wxString& name)
 {
+    if ( !wxControl::Create(parent, id, pos, size,
+                            style | wxCLIP_CHILDREN | wxWANTS_CHARS,
+                            wxDefaultValidator, name) )
+    {
+        return FALSE;
+    }
+
     // needed to get the arrow keys normally used for the dialog navigation
     SetWindowStyle(style | wxWANTS_CHARS);
 
     m_date = date.IsValid() ? date : wxDateTime::Today();
 
-    m_spinYear = new wxYearSpinCtrl(this);
-    m_staticYear = new wxStaticText(GetParent(), -1, m_date.Format(_T("%Y")),
-                                    wxDefaultPosition, wxDefaultSize,
-                                    wxALIGN_CENTRE);
+    m_lowdate = wxDefaultDateTime;
+    m_highdate = wxDefaultDateTime;
 
-    m_comboMonth = new wxMonthComboBox(this);
-    m_staticMonth = new wxStaticText(GetParent(), -1, m_date.Format(_T("%B")),
-                                     wxDefaultPosition, wxDefaultSize,
-                                     wxALIGN_CENTRE);
+    if ( !(GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) )
+    {
+        m_spinYear = new wxYearSpinCtrl(this);
+        m_staticYear = new wxStaticText(GetParent(), -1, m_date.Format(_T("%Y")),
+                                        wxDefaultPosition, wxDefaultSize,
+                                        wxALIGN_CENTRE);
+
+        m_comboMonth = new wxMonthComboBox(this);
+        m_staticMonth = new wxStaticText(GetParent(), -1, m_date.Format(_T("%B")),
+                                         wxDefaultPosition, wxDefaultSize,
+                                         wxALIGN_CENTRE);
+    }
 
     ShowCurrentControls();
 
@@ -236,8 +260,14 @@ bool wxCalendarCtrl::Show(bool show)
         return FALSE;
     }
 
-    GetMonthControl()->Show(show);
-    GetYearControl()->Show(show);
+    if ( !(GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) )
+    {
+        if ( GetMonthControl() )
+        {
+            GetMonthControl()->Show(show);
+            GetYearControl()->Show(show);
+        }
+    }
 
     return TRUE;
 }
@@ -249,8 +279,11 @@ bool wxCalendarCtrl::Enable(bool enable)
         return FALSE;
     }
 
-    GetMonthControl()->Enable(enable);
-    GetYearControl()->Enable(enable);
+    if ( !(GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) )
+    {
+        GetMonthControl()->Enable(enable);
+        GetYearControl()->Enable(enable);
+    }
 
     return TRUE;
 }
@@ -261,39 +294,42 @@ bool wxCalendarCtrl::Enable(bool enable)
 
 void wxCalendarCtrl::ShowCurrentControls()
 {
-    if ( AllowMonthChange() )
+    if ( !(GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) )
     {
-        m_comboMonth->Show();
-        m_staticMonth->Hide();
-
-        if ( AllowYearChange() )
+        if ( AllowMonthChange() )
         {
-            m_spinYear->Show();
-            m_staticYear->Hide();
+            m_comboMonth->Show();
+            m_staticMonth->Hide();
+
+            if ( AllowYearChange() )
+            {
+                m_spinYear->Show();
+                m_staticYear->Hide();
 
-            // skip the rest
-            return;
+                // skip the rest
+                return;
+            }
+        }
+        else
+        {
+            m_comboMonth->Hide();
+            m_staticMonth->Show();
         }
-    }
-    else
-    {
-        m_comboMonth->Hide();
-        m_staticMonth->Show();
-    }
 
-    // year change not allowed here
-    m_spinYear->Hide();
-    m_staticYear->Show();
+        // year change not allowed here
+        m_spinYear->Hide();
+        m_staticYear->Show();
+    }
 }
 
 wxControl *wxCalendarCtrl::GetMonthControl() const
 {
-    return AllowMonthChange() ? m_comboMonth : m_staticMonth;
+    return AllowMonthChange() ? (wxControl *)m_comboMonth : (wxControl *)m_staticMonth;
 }
 
 wxControl *wxCalendarCtrl::GetYearControl() const
 {
-    return AllowYearChange() ? m_spinYear : m_staticYear;
+    return AllowYearChange() ? (wxControl *)m_spinYear : (wxControl *)m_staticYear;
 }
 
 void wxCalendarCtrl::EnableYearChange(bool enable)
@@ -308,6 +344,10 @@ void wxCalendarCtrl::EnableYearChange(bool enable)
         SetWindowStyle(style);
 
         ShowCurrentControls();
+        if ( GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION )
+        {
+            Refresh();
+        }
     }
 }
 
@@ -323,6 +363,10 @@ void wxCalendarCtrl::EnableMonthChange(bool enable)
         SetWindowStyle(style);
 
         ShowCurrentControls();
+        if ( GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION )
+        {
+            Refresh();
+        }
     }
 }
 
@@ -330,38 +374,53 @@ void wxCalendarCtrl::EnableMonthChange(bool enable)
 // changing date
 // ----------------------------------------------------------------------------
 
-void wxCalendarCtrl::SetDate(const wxDateTime& date)
+bool wxCalendarCtrl::SetDate(const wxDateTime& date)
 {
+    bool retval = FALSE;
+
     bool sameMonth = m_date.GetMonth() == date.GetMonth(),
          sameYear = m_date.GetYear() == date.GetYear();
 
-    if ( sameMonth && sameYear )
+    if ( IsDateInRange(date) )
     {
-        // just change the day
-        ChangeDay(date);
-    }
-    else
-    {
-        if ( !AllowMonthChange() || (!AllowYearChange() && !sameYear) )
+        if ( sameMonth && sameYear )
         {
-            // forbidden
-            return;
+            // just change the day
+            retval = TRUE;
+            ChangeDay(date);
         }
+        else
+        {
+            if ( !AllowMonthChange() || (!AllowYearChange() && !sameYear) )
+            {
+                // forbidden
+                return retval;
+            }
 
-        // change everything
-        m_date = date;
+            // change everything
+            retval = TRUE;
+            m_date = date;
 
-        // update the controls
-        m_comboMonth->SetSelection(m_date.GetMonth());
+            if ( !(GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) )
+            {
+                // update the controls
+                m_comboMonth->SetSelection(m_date.GetMonth());
 
-        if ( AllowYearChange() )
-        {
-            m_spinYear->SetValue(m_date.Format(_T("%Y")));
-        }
+                if ( AllowYearChange() )
+                {
+                    m_spinYear->SetValue(m_date.Format(_T("%Y")));
+                }
+            }
 
-        // update the calendar
-        Refresh();
+            // as the month changed, holidays did too
+            SetHolidayAttrs();
+
+            // update the calendar
+            Refresh();
+        }
     }
+
+    return retval;
 }
 
 void wxCalendarCtrl::ChangeDay(const wxDateTime& date)
@@ -398,9 +457,65 @@ void wxCalendarCtrl::SetDateAndNotify(const wxDateTime& date)
     else
         return;
 
-    SetDate(date);
+    if ( SetDate(date) )
+    {
+        GenerateEvents(type, wxEVT_CALENDAR_SEL_CHANGED);
+    }
+}
+
+// ----------------------------------------------------------------------------
+// date range
+// ----------------------------------------------------------------------------
+
+bool wxCalendarCtrl::SetLowerDateLimit(const wxDateTime& date /* = wxDefaultDateTime */)
+{
+    bool retval = TRUE;
+
+    if ( !(date.IsValid()) || ( ( m_highdate.IsValid() ) ? ( date <= m_highdate ) : TRUE ) )
+    {
+        m_lowdate = date;
+    }
+    else
+    {
+        retval = FALSE;
+    }
 
-    GenerateEvents(type, wxEVT_CALENDAR_SEL_CHANGED);
+    return retval;
+}
+
+bool wxCalendarCtrl::SetUpperDateLimit(const wxDateTime& date /* = wxDefaultDateTime */)
+{
+    bool retval = TRUE;
+
+    if ( !(date.IsValid()) || ( ( m_lowdate.IsValid() ) ? ( date >= m_lowdate ) : TRUE ) )
+    {
+        m_highdate = date;
+    }
+    else
+    {
+        retval = FALSE;
+    }
+
+    return retval;
+}
+
+bool wxCalendarCtrl::SetDateRange(const wxDateTime& lowerdate /* = wxDefaultDateTime */, const wxDateTime& upperdate /* = wxDefaultDateTime */)
+{
+    bool retval = TRUE;
+
+    if (
+        ( !( lowerdate.IsValid() ) || ( ( upperdate.IsValid() ) ? ( lowerdate <= upperdate ) : TRUE ) ) &&
+        ( !( upperdate.IsValid() ) || ( ( lowerdate.IsValid() ) ? ( upperdate >= lowerdate ) : TRUE ) ) )
+    {
+        m_lowdate = lowerdate;
+        m_highdate = upperdate;
+    }
+    else
+    {
+        retval = FALSE;
+    }
+
+    return retval;
 }
 
 // ----------------------------------------------------------------------------
@@ -417,19 +532,123 @@ wxDateTime wxCalendarCtrl::GetStartDate() const
     date.SetToPrevWeekDay(GetWindowStyle() & wxCAL_MONDAY_FIRST
                           ? wxDateTime::Mon : wxDateTime::Sun);
 
+    if ( GetWindowStyle() & wxCAL_SHOW_SURROUNDING_WEEKS )
+    {
+        // We want to offset the calendar if we start on the first..
+        if ( date.GetDay() == 1 )
+        {
+            date -= wxDateSpan::Week();
+        }
+    }
+
     return date;
 }
 
 bool wxCalendarCtrl::IsDateShown(const wxDateTime& date) const
 {
-    return date.GetMonth() == m_date.GetMonth();
+    if ( !(GetWindowStyle() & wxCAL_SHOW_SURROUNDING_WEEKS) )
+    {
+        return date.GetMonth() == m_date.GetMonth();
+    }
+    else
+    {
+        return TRUE;
+    }
+}
+
+bool wxCalendarCtrl::IsDateInRange(const wxDateTime& date) const
+{
+    bool retval = TRUE;
+    // Check if the given date is in the range specified
+    retval = ( ( ( m_lowdate.IsValid() ) ? ( date >= m_lowdate ) : TRUE )
+        && ( ( m_highdate.IsValid() ) ? ( date <= m_highdate ) : TRUE ) );
+    return retval;
+}
+
+bool wxCalendarCtrl::ChangeYear(wxDateTime* target) const
+{
+    bool retval = FALSE;
+
+    if ( !(IsDateInRange(*target)) )
+    {
+        if ( target->GetYear() < m_date.GetYear() )
+        {
+            if ( target->GetYear() >= GetLowerDateLimit().GetYear() )
+            {
+                *target = GetLowerDateLimit();
+                retval = TRUE;
+            }
+            else
+            {
+                *target = m_date;
+            }
+        }
+        else
+        {
+            if ( target->GetYear() <= GetUpperDateLimit().GetYear() )
+            {
+                *target = GetUpperDateLimit();
+                retval = TRUE;
+            }
+            else
+            {
+                *target = m_date;
+            }
+        }
+    }
+    else
+    {
+        retval = TRUE;
+    }
+
+    return retval;
+}
+
+bool wxCalendarCtrl::ChangeMonth(wxDateTime* target) const
+{
+    bool retval = TRUE;
+
+    if ( !(IsDateInRange(*target)) )
+    {
+        retval = FALSE;
+
+        if ( target->GetMonth() < m_date.GetMonth() )
+        {
+            *target = GetLowerDateLimit();
+        }
+        else
+        {
+            *target = GetUpperDateLimit();
+        }
+    }
+
+    return retval;
 }
 
 size_t wxCalendarCtrl::GetWeek(const wxDateTime& date) const
 {
-    return date.GetWeekOfMonth(GetWindowStyle() & wxCAL_MONDAY_FIRST
-                               ? wxDateTime::Monday_First
-                               : wxDateTime::Sunday_First);
+    size_t retval = date.GetWeekOfMonth(GetWindowStyle() & wxCAL_MONDAY_FIRST
+                                   ? wxDateTime::Monday_First
+                                   : wxDateTime::Sunday_First);
+
+    if ( (GetWindowStyle() & wxCAL_SHOW_SURROUNDING_WEEKS) )
+    {
+        // we need to offset an extra week if we "start" on the 1st of the month
+        wxDateTime::Tm tm = date.GetTm();
+
+        wxDateTime datetest = wxDateTime(1, tm.mon, tm.year);
+
+        // rewind back
+        datetest.SetToPrevWeekDay(GetWindowStyle() & wxCAL_MONDAY_FIRST
+                              ? wxDateTime::Mon : wxDateTime::Sun);
+
+        if ( datetest.GetDay() == 1 )
+        {
+            retval += 1;
+        }
+    }
+
+    return retval;
 }
 
 // ----------------------------------------------------------------------------
@@ -454,17 +673,26 @@ wxSize wxCalendarCtrl::DoGetBestSize() const
     ((wxCalendarCtrl *)this)->RecalcGeometry(); // const_cast
 
     wxCoord width = 7*m_widthCol,
-            height = 7*m_heightRow;
+            height = 7*m_heightRow + m_rowOffset;
 
-    wxSize sizeCombo = m_comboMonth->GetBestSize(),
-           sizeSpin = m_spinYear->GetBestSize();
+    // the combobox doesn't report its height correctly (it returns the
+    // height including the drop down list) so don't use it
 
-    height += VERT_MARGIN + wxMax(sizeCombo.y, sizeSpin.y);
+    if ( !(GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) )
+    {
+        height += VERT_MARGIN + m_spinYear->GetBestSize().y;
+    }
+    else
+    {
+        height += VERT_MARGIN;
+    }
 
-    if ( GetWindowStyle() & (wxRAISED_BORDER | wxSUNKEN_BORDER) )
+//    if ( GetWindowStyle() & (wxRAISED_BORDER | wxSUNKEN_BORDER) ) // This doesn't work. Default is wxBORDER_DEFAULT (0)
+    if ( !(GetWindowStyle() & wxBORDER_NONE) )
     {
         // the border would clip the last line otherwise
-        height += 4;
+        height += 6;
+        width += 4;
     }
 
     return wxSize(width, height);
@@ -479,19 +707,27 @@ void wxCalendarCtrl::DoSetSize(int x, int y,
 
 void wxCalendarCtrl::DoMoveWindow(int x, int y, int width, int height)
 {
-    wxSize sizeCombo = m_comboMonth->GetSize();
-    wxSize sizeStatic = m_staticMonth->GetSize();
+    int xDiff = 0;
+    int yDiff = 0;
 
-    int dy = (sizeCombo.y - sizeStatic.y) / 2;
-    m_comboMonth->Move(x, y);
-    m_staticMonth->SetSize(x, y + dy, sizeCombo.x, sizeStatic.y);
+    if ( !(GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) )
+    {
+        wxSize sizeCombo = m_comboMonth->GetSize();
+        wxSize sizeStatic = m_staticMonth->GetSize();
 
-    int xDiff = sizeCombo.x + HORZ_MARGIN;
-    m_spinYear->SetSize(x + xDiff, y, width - xDiff, sizeCombo.y);
-    m_staticYear->SetSize(x + xDiff, y + dy, width - xDiff, sizeStatic.y);
+        int dy = (sizeCombo.y - sizeStatic.y) / 2;
 
-    wxSize sizeSpin = m_spinYear->GetSize();
-    int yDiff = wxMax(sizeSpin.y, sizeCombo.y) + VERT_MARGIN;
+        m_comboMonth->Move(x, y);
+        m_staticMonth->SetSize(x, y + dy, sizeCombo.x, sizeStatic.y);
+
+        xDiff = sizeCombo.x + HORZ_MARGIN;
+
+        m_spinYear->SetSize(x + xDiff, y, width - xDiff, sizeCombo.y);
+        m_staticYear->SetSize(x + xDiff, y + dy, width - xDiff, sizeStatic.y);
+
+        wxSize sizeSpin = m_spinYear->GetSize();
+        yDiff = wxMax(sizeSpin.y, sizeCombo.y) + VERT_MARGIN;
+    }
 
     wxControl::DoMoveWindow(x, y + yDiff, width, height - yDiff);
 }
@@ -500,10 +736,13 @@ void wxCalendarCtrl::DoGetPosition(int *x, int *y) const
 {
     wxControl::DoGetPosition(x, y);
 
-    // our real top corner is not in this position
-    if ( y )
+    if ( !(GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) )
     {
-        *y -= GetMonthControl()->GetSize().y + VERT_MARGIN;
+        // our real top corner is not in this position
+        if ( y )
+        {
+            *y -= GetMonthControl()->GetSize().y + VERT_MARGIN;
+        }
     }
 }
 
@@ -511,10 +750,13 @@ void wxCalendarCtrl::DoGetSize(int *width, int *height) const
 {
     wxControl::DoGetSize(width, height);
 
-    // our real height is bigger
-    if ( height )
+    if ( !(GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) )
     {
-        *height += GetMonthControl()->GetSize().y + VERT_MARGIN;
+        // our real height is bigger
+        if ( height )
+        {
+            *height += GetMonthControl()->GetSize().y + VERT_MARGIN;
+        }
     }
 }
 
@@ -544,6 +786,8 @@ void wxCalendarCtrl::RecalcGeometry()
     // leave some margins
     m_widthCol += 2;
     m_heightRow += 2;
+
+    m_rowOffset = (GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) ? m_heightRow : 0; // conditional in relation to style
 }
 
 // ----------------------------------------------------------------------------
@@ -559,23 +803,99 @@ void wxCalendarCtrl::OnPaint(wxPaintEvent& WXUNUSED(event))
     RecalcGeometry();
 
 #if DEBUG_PAINT
-    printf("--- starting to paint, selection: %s, week %u\n",
+    wxLogDebug("--- starting to paint, selection: %s, week %u\n",
            m_date.Format("%a %d-%m-%Y %H:%M:%S").c_str(),
            GetWeek(m_date));
 #endif
 
+    wxCoord y = 0;
+
+/////////////////////////////////////////////////////////////////////////////////////////
+
+    if ( (GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) )
+    {
+        // draw the sequential month-selector
+
+        dc.SetBackgroundMode(wxTRANSPARENT);
+        dc.SetTextForeground(*wxBLACK);
+        dc.SetBrush(wxBrush(m_colHeaderBg, wxSOLID));
+        dc.SetPen(wxPen(m_colHeaderBg, 1, wxSOLID));
+        dc.DrawRectangle(0, y, 7*m_widthCol, m_heightRow);
+
+        // Get extent of month-name + year
+        wxCoord monthw, monthh;
+        wxString headertext = m_date.Format(wxT("%B %Y"));
+        dc.GetTextExtent(headertext, &monthw, &monthh);
+
+        // draw month-name centered above weekdays
+        wxCoord monthx = ((m_widthCol * 7) - monthw) / 2;
+        wxCoord monthy = ((m_heightRow - monthh) / 2) + y;
+        dc.DrawText(headertext, monthx,  monthy);
+
+        // calculate the "month-arrows"
+        wxPoint leftarrow[3];
+        wxPoint rightarrow[3];
+
+        int arrowheight = monthh / 2;
+
+        leftarrow[0] = wxPoint(0, arrowheight / 2);
+        leftarrow[1] = wxPoint(arrowheight / 2, 0);
+        leftarrow[2] = wxPoint(arrowheight / 2, arrowheight - 1);
+
+        rightarrow[0] = wxPoint(0, 0);
+        rightarrow[1] = wxPoint(arrowheight / 2, arrowheight / 2);
+        rightarrow[2] = wxPoint(0, arrowheight - 1);
+
+        // draw the "month-arrows"
+
+        wxCoord arrowy = (m_heightRow - arrowheight) / 2;
+        wxCoord larrowx = (m_widthCol - (arrowheight / 2)) / 2;
+        wxCoord rarrowx = ((m_widthCol - (arrowheight / 2)) / 2) + m_widthCol*6;
+        m_leftArrowRect = wxRect(0, 0, 0, 0);
+        m_rightArrowRect = wxRect(0, 0, 0, 0);
+
+        if ( AllowMonthChange() )
+        {
+            wxDateTime ldpm = wxDateTime(1,m_date.GetMonth(), m_date.GetYear()) - wxDateSpan::Day(); // last day prev month
+            // Check if range permits change
+            if ( IsDateInRange(ldpm) && ( ( ldpm.GetYear() == m_date.GetYear() ) ? TRUE : AllowYearChange() ) )
+            {
+                m_leftArrowRect = wxRect(larrowx - 3, arrowy - 3, (arrowheight / 2) + 8, (arrowheight + 6));
+                dc.SetBrush(wxBrush(*wxBLACK, wxSOLID));
+                dc.SetPen(wxPen(*wxBLACK, 1, wxSOLID));
+                dc.DrawPolygon(3, leftarrow, larrowx , arrowy, wxWINDING_RULE);
+                dc.SetBrush(*wxTRANSPARENT_BRUSH);
+                dc.DrawRectangle(m_leftArrowRect);
+            }
+            wxDateTime fdnm = wxDateTime(1,m_date.GetMonth(), m_date.GetYear()) + wxDateSpan::Month(); // first day next month
+            if ( IsDateInRange(fdnm) && ( ( fdnm.GetYear() == m_date.GetYear() ) ? TRUE : AllowYearChange() ) )
+            {
+                m_rightArrowRect = wxRect(rarrowx - 4, arrowy - 3, (arrowheight / 2) + 8, (arrowheight + 6));
+                dc.SetBrush(wxBrush(*wxBLACK, wxSOLID));
+                dc.SetPen(wxPen(*wxBLACK, 1, wxSOLID));
+                dc.DrawPolygon(3, rightarrow, rarrowx , arrowy, wxWINDING_RULE);
+                dc.SetBrush(*wxTRANSPARENT_BRUSH);
+                dc.DrawRectangle(m_rightArrowRect);
+            }
+        }
+
+        y += m_heightRow;
+    }
+
+/////////////////////////////////////////////////////////////////////////////////////////
+
     // first draw the week days
-    if ( IsExposed(0, 0, 7*m_widthCol, m_heightRow) )
+    if ( IsExposed(0, y, 7*m_widthCol, m_heightRow) )
     {
 #if DEBUG_PAINT
-        puts("painting the header");
+        wxLogDebug("painting the header");
 #endif
 
         dc.SetBackgroundMode(wxTRANSPARENT);
         dc.SetTextForeground(m_colHeaderFg);
         dc.SetBrush(wxBrush(m_colHeaderBg, wxSOLID));
         dc.SetPen(wxPen(m_colHeaderBg, 1, wxSOLID));
-        dc.DrawRectangle(0, 0, 7*m_widthCol, m_heightRow);
+        dc.DrawRectangle(0, y, 7*m_widthCol, m_heightRow);
 
         bool startOnMonday = (GetWindowStyle() & wxCAL_MONDAY_FIRST) != 0;
         for ( size_t wd = 0; wd < 7; wd++ )
@@ -585,8 +905,10 @@ void wxCalendarCtrl::OnPaint(wxPaintEvent& WXUNUSED(event))
                 n = wd == 6 ? 0 : wd + 1;
             else
                 n = wd;
-
-            dc.DrawText(m_weekdays[n], wd*m_widthCol + 1, 0);
+            wxCoord dayw, dayh;
+            dc.GetTextExtent(m_weekdays[n], &dayw, &dayh);
+//            dc.DrawText(m_weekdays[n], wd*m_widthCol + 1, y);
+            dc.DrawText(m_weekdays[n], (wd*m_widthCol) + ((m_widthCol- dayw) / 2), y); // center the day-name
         }
     }
 
@@ -594,11 +916,11 @@ void wxCalendarCtrl::OnPaint(wxPaintEvent& WXUNUSED(event))
     dc.SetTextForeground(*wxBLACK);
     //dc.SetFont(*wxNORMAL_FONT);
 
-    wxCoord y = m_heightRow;
-
+    y += m_heightRow;
     wxDateTime date = GetStartDate();
+
 #if DEBUG_PAINT
-    printf("starting calendar from %s\n",
+    wxLogDebug("starting calendar from %s\n",
             date.Format("%a %d-%m-%Y %H:%M:%S").c_str());
 #endif
 
@@ -614,7 +936,7 @@ void wxCalendarCtrl::OnPaint(wxPaintEvent& WXUNUSED(event))
         }
 
 #if DEBUG_PAINT
-        printf("painting week %d at y = %d\n", nWeek, y);
+        wxLogDebug("painting week %d at y = %d\n", nWeek, y);
 #endif
 
         for ( size_t wd = 0; wd < 7; wd++ )
@@ -630,47 +952,60 @@ void wxCalendarCtrl::OnPaint(wxPaintEvent& WXUNUSED(event))
                 bool changedColours = FALSE,
                      changedFont = FALSE;
 
-                wxCalendarDateAttr *attr = m_attrs[day - 1];
+                bool isSel = FALSE;
+                wxCalendarDateAttr *attr = NULL;
 
-                bool isSel = m_date == date;
-                if ( isSel )
+                if ( date.GetMonth() != m_date.GetMonth() || !IsDateInRange(date) )
                 {
-                    dc.SetTextForeground(m_colHighlightFg);
-                    dc.SetTextBackground(m_colHighlightBg);
-
+                    // surrounding week or out-of-range
+                    // draw "disabled"
+                    dc.SetTextForeground(*wxLIGHT_GREY);
                     changedColours = TRUE;
                 }
-                else if ( attr )
+                else
                 {
-                    wxColour colFg, colBg;
-
-                    if ( attr->IsHoliday() )
-                    {
-                        colFg = m_colHolidayFg;
-                        colBg = m_colHolidayBg;
-                    }
-                    else
-                    {
-                        colFg = attr->GetTextColour();
-                        colBg = attr->GetBackgroundColour();
-                    }
+                    isSel = date.IsSameDate(m_date);
+                    attr = m_attrs[day - 1];
 
-                    if ( colFg.Ok() )
+                    if ( isSel )
                     {
-                        dc.SetTextForeground(colFg);
-                        changedColours = TRUE;
-                    }
+                        dc.SetTextForeground(m_colHighlightFg);
+                        dc.SetTextBackground(m_colHighlightBg);
 
-                    if ( colBg.Ok() )
-                    {
-                        dc.SetTextBackground(colBg);
                         changedColours = TRUE;
                     }
-
-                    if ( attr->HasFont() )
+                    else if ( attr )
                     {
-                        dc.SetFont(attr->GetFont());
-                        changedFont = TRUE;
+                        wxColour colFg, colBg;
+
+                        if ( attr->IsHoliday() )
+                        {
+                            colFg = m_colHolidayFg;
+                            colBg = m_colHolidayBg;
+                        }
+                        else
+                        {
+                            colFg = attr->GetTextColour();
+                            colBg = attr->GetBackgroundColour();
+                        }
+
+                        if ( colFg.Ok() )
+                        {
+                            dc.SetTextForeground(colFg);
+                            changedColours = TRUE;
+                        }
+
+                        if ( colBg.Ok() )
+                        {
+                            dc.SetTextBackground(colBg);
+                            changedColours = TRUE;
+                        }
+
+                        if ( attr->HasFont() )
+                        {
+                            dc.SetFont(attr->GetFont());
+                            changedFont = TRUE;
+                        }
                     }
                 }
 
@@ -726,8 +1061,34 @@ void wxCalendarCtrl::OnPaint(wxPaintEvent& WXUNUSED(event))
             date += wxDateSpan::Day();
         }
     }
+
+    // Greying out out-of-range background
+    bool showSurrounding = (GetWindowStyle() & wxCAL_SHOW_SURROUNDING_WEEKS) != 0;
+
+    date = ( showSurrounding ) ? GetStartDate() : wxDateTime(1, m_date.GetMonth(), m_date.GetYear());
+    if ( !IsDateInRange(date) )
+    {
+        wxDateTime firstOOR = GetLowerDateLimit() - wxDateSpan::Day(); // first out-of-range
+
+        wxBrush oorbrush = *wxLIGHT_GREY_BRUSH;
+        oorbrush.SetStyle(wxFDIAGONAL_HATCH);
+
+        HighlightRange(&dc, date, firstOOR, wxTRANSPARENT_PEN, &oorbrush);
+    }
+
+    date = ( showSurrounding ) ? GetStartDate() + wxDateSpan::Weeks(6) - wxDateSpan::Day() : wxDateTime().SetToLastMonthDay(m_date.GetMonth(), m_date.GetYear());
+    if ( !IsDateInRange(date) )
+    {
+        wxDateTime firstOOR = GetUpperDateLimit() + wxDateSpan::Day(); // first out-of-range
+
+        wxBrush oorbrush = *wxLIGHT_GREY_BRUSH;
+        oorbrush.SetStyle(wxFDIAGONAL_HATCH);
+
+        HighlightRange(&dc, firstOOR, date, wxTRANSPARENT_PEN, &oorbrush);
+    }
+
 #if DEBUG_PAINT
-    puts("+++ finished painting");
+    wxLogDebug("+++ finished painting");
 #endif
 }
 
@@ -741,12 +1102,23 @@ void wxCalendarCtrl::RefreshDate(const wxDateTime& date)
     // the whole row anyhow - and this allows the small optimisation in
     // OnClick() below to work
     rect.x = 0;
-    rect.y = m_heightRow * GetWeek(date);
+
+    rect.y = (m_heightRow * GetWeek(date)) + m_rowOffset;
+
     rect.width = 7*m_widthCol;
     rect.height = m_heightRow;
 
+#ifdef __WXMSW__
+    // VZ: for some reason, the selected date seems to occupy more space under
+    //     MSW - this is probably some bug in the font size calculations, but I
+    //     don't know where exactly. This fix is ugly and leads to more
+    //     refreshes than really needed, but without it the selected days
+    //     leaves even more ugly underscores on screen.
+    rect.Inflate(0, 1);
+#endif // MSW
+
 #if DEBUG_PAINT
-    printf("*** refreshing week %d at (%d, %d)-(%d, %d)\n",
+    wxLogDebug("*** refreshing week %d at (%d, %d)-(%d, %d)\n",
            GetWeek(date),
            rect.x, rect.y,
            rect.x + rect.width, rect.y + rect.height);
@@ -755,6 +1127,184 @@ void wxCalendarCtrl::RefreshDate(const wxDateTime& date)
     Refresh(TRUE, &rect);
 }
 
+void wxCalendarCtrl::HighlightRange(wxPaintDC* pDC, const wxDateTime& fromdate, const wxDateTime& todate, wxPen* pPen, wxBrush* pBrush)
+{
+    // Highlights the given range using pen and brush
+    // Does nothing if todate < fromdate
+
+
+#if DEBUG_PAINT
+    wxLogDebug("+++ HighlightRange: (%s) - (%s) +++", fromdate.Format("%d %m %Y"), todate.Format("%d %m %Y"));
+#endif
+
+    if ( todate >= fromdate )
+    {
+        // do stuff
+        // date-coordinates
+        int fd, fw;
+        int td, tw;
+
+        // implicit: both dates must be currently shown - checked by GetDateCoord
+        if ( GetDateCoord(fromdate, &fd, &fw) && GetDateCoord(todate, &td, &tw) )
+        {
+#if DEBUG_PAINT
+            wxLogDebug("Highlight range: (%i, %i) - (%i, %i)", fd, fw, td, tw);
+#endif
+            if ( ( (tw - fw) == 1 ) && ( td < fd ) )
+            {
+                // special case: interval 7 days or less not in same week
+                // split in two seperate intervals
+                wxDateTime tfd = fromdate + wxDateSpan::Days(7-fd);
+                wxDateTime ftd = tfd + wxDateSpan::Day();
+#if DEBUG_PAINT
+                wxLogDebug("Highlight: Seperate segments");
+#endif
+                // draw seperately
+                HighlightRange(pDC, fromdate, tfd, pPen, pBrush);
+                HighlightRange(pDC, ftd, todate, pPen, pBrush);
+            }
+            else
+            {
+                int numpoints;
+                wxPoint corners[8]; // potentially 8 corners in polygon
+
+                if ( fw == tw )
+                {
+                    // simple case: same week
+                    numpoints = 4;
+                    corners[0] = wxPoint((fd - 1) * m_widthCol, (fw * m_heightRow) + m_rowOffset);
+                    corners[1] = wxPoint((fd - 1) * m_widthCol, ((fw + 1 ) * m_heightRow) + m_rowOffset);
+                    corners[2] = wxPoint(td * m_widthCol, ((tw + 1) * m_heightRow) + m_rowOffset);
+                    corners[3] = wxPoint(td * m_widthCol, (tw * m_heightRow) + m_rowOffset);
+                }
+                else
+                {
+                    int cidx = 0;
+                    // "complex" polygon
+                    corners[cidx] = wxPoint((fd - 1) * m_widthCol, (fw * m_heightRow) + m_rowOffset); cidx++;
+
+                    if ( fd > 1 )
+                    {
+                        corners[cidx] = wxPoint((fd - 1) * m_widthCol, ((fw + 1) * m_heightRow) + m_rowOffset); cidx++;
+                        corners[cidx] = wxPoint(0, ((fw + 1) * m_heightRow) + m_rowOffset); cidx++;
+                    }
+
+                    corners[cidx] = wxPoint(0, ((tw + 1) * m_heightRow) + m_rowOffset); cidx++;
+                    corners[cidx] = wxPoint(td * m_widthCol, ((tw + 1) * m_heightRow) + m_rowOffset); cidx++;
+
+                    if ( td < 7 )
+                    {
+                        corners[cidx] = wxPoint(td * m_widthCol, (tw * m_heightRow) + m_rowOffset); cidx++;
+                        corners[cidx] = wxPoint(7 * m_widthCol, (tw * m_heightRow) + m_rowOffset); cidx++;
+                    }
+
+                    corners[cidx] = wxPoint(7 * m_widthCol, (fw * m_heightRow) + m_rowOffset); cidx++;
+
+                    numpoints = cidx;
+                }
+
+                // draw the polygon
+                pDC->SetBrush(*pBrush);
+                pDC->SetPen(*pPen);
+                pDC->DrawPolygon(numpoints, corners);
+            }
+        }
+    }
+    // else do nothing
+#if DEBUG_PAINT
+    wxLogDebug("--- HighlightRange ---");
+#endif
+}
+
+bool wxCalendarCtrl::GetDateCoord(const wxDateTime& date, int *day, int *week) const
+{
+    bool retval = TRUE;
+
+#if DEBUG_PAINT
+    wxLogDebug("+++ GetDateCoord: (%s) +++", date.Format("%d %m %Y"));
+#endif
+
+    if ( IsDateShown(date) )
+    {
+        bool startOnMonday = ( GetWindowStyle() & wxCAL_MONDAY_FIRST ) != 0;
+
+        // Find day
+        *day = date.GetWeekDay();
+
+        if ( *day == 0 ) // sunday
+        {
+            *day = ( startOnMonday ) ? 7 : 1;
+        }
+        else
+        {
+            day += ( startOnMonday ) ? 0 : 1;
+        }
+
+        int targetmonth = date.GetMonth() + (12 * date.GetYear());
+        int thismonth = m_date.GetMonth() + (12 * m_date.GetYear());
+
+        // Find week
+        if ( targetmonth == thismonth )
+        {
+            *week = GetWeek(date);
+        }
+        else
+        {
+            if ( targetmonth < thismonth )
+            {
+                *week = 1; // trivial
+            }
+            else // targetmonth > thismonth
+            {
+                wxDateTime ldcm;
+                int lastweek;
+                int lastday;
+
+                // get the datecoord of the last day in the month currently shown
+#if DEBUG_PAINT
+                wxLogDebug("     +++ LDOM +++");
+#endif
+                GetDateCoord(ldcm.SetToLastMonthDay(m_date.GetMonth(), m_date.GetYear()), &lastday, &lastweek);
+#if DEBUG_PAINT
+                wxLogDebug("     --- LDOM ---");
+#endif
+
+                wxTimeSpan span = date - ldcm;
+
+                int daysfromlast = span.GetDays();
+#if DEBUG_PAINT
+                wxLogDebug("daysfromlast: %i", daysfromlast);
+#endif
+                if ( daysfromlast + lastday > 7 ) // past week boundary
+                {
+                    int wholeweeks = (daysfromlast / 7);
+                    *week = wholeweeks + lastweek;
+                    if ( (daysfromlast - (7 * wholeweeks) + lastday) > 7 )
+                    {
+                        *week += 1;
+                    }
+                }
+                else
+                {
+                    *week = lastweek;
+                }
+            }
+        }
+    }
+    else
+    {
+        *day = -1;
+        *week = -1;
+        retval = FALSE;
+    }
+
+#if DEBUG_PAINT
+    wxLogDebug("--- GetDateCoord: (%s) = (%i, %i) ---", date.Format("%d %m %Y"), *day, *week);
+#endif
+
+    return retval;
+}
+
 // ----------------------------------------------------------------------------
 // mouse handling
 // ----------------------------------------------------------------------------
@@ -778,10 +1328,13 @@ void wxCalendarCtrl::OnClick(wxMouseEvent& event)
     switch ( HitTest(event.GetPosition(), &date, &wday) )
     {
         case wxCAL_HITTEST_DAY:
-            ChangeDay(date);
+            if ( IsDateInRange(date) )
+            {
+                ChangeDay(date);
 
-            GenerateEvents(wxEVT_CALENDAR_DAY_CHANGED,
-                           wxEVT_CALENDAR_SEL_CHANGED);
+                GenerateEvents(wxEVT_CALENDAR_DAY_CHANGED,
+                               wxEVT_CALENDAR_SEL_CHANGED);
+            }
             break;
 
         case wxCAL_HITTEST_HEADER:
@@ -792,6 +1345,12 @@ void wxCalendarCtrl::OnClick(wxMouseEvent& event)
             }
             break;
 
+        case wxCAL_HITTEST_DECMONTH:
+        case wxCAL_HITTEST_INCMONTH:
+        case wxCAL_HITTEST_SURROUNDING_WEEK:
+            SetDateAndNotify(date); // we probably only want to refresh the control. No notification.. (maybe as an option?)
+            break;
+
         default:
             wxFAIL_MSG(_T("unknown hittest code"));
             // fall through
@@ -808,25 +1367,79 @@ wxCalendarHitTestResult wxCalendarCtrl::HitTest(const wxPoint& pos,
 {
     RecalcGeometry();
 
-    int wday = pos.x / m_widthCol;
-
     wxCoord y = pos.y;
-    if ( y < m_heightRow )
+
+///////////////////////////////////////////////////////////////////////////////////////////////////////
+    if ( (GetWindowStyle() & wxCAL_SEQUENTIAL_MONTH_SELECTION) )
     {
-        if ( wd )
+        // Header: month
+
+        // we need to find out if the hit is on left arrow, on month or on right arrow
+        // left arrow?
+        if ( wxRegion(m_leftArrowRect).Contains(pos) == wxInRegion )
         {
-            if ( GetWindowStyle() & wxCAL_MONDAY_FIRST )
+            if ( date )
             {
-                wday = wday == 6 ? 0 : wday + 1;
+                if ( IsDateInRange(m_date - wxDateSpan::Month()) )
+                {
+                    *date = m_date - wxDateSpan::Month();
+                }
+                else
+                {
+                    *date = GetLowerDateLimit();
+                }
+            }
+
+            return wxCAL_HITTEST_DECMONTH;
+        }
+
+        if ( wxRegion(m_rightArrowRect).Contains(pos) == wxInRegion )
+        {
+            if ( date )
+            {
+                if ( IsDateInRange(m_date + wxDateSpan::Month()) )
+                {
+                    *date = m_date + wxDateSpan::Month();
+                }
+                else
+                {
+                    *date = GetUpperDateLimit();
+                }
             }
 
-            *wd = (wxDateTime::WeekDay)wday;
+            return wxCAL_HITTEST_INCMONTH;
         }
 
-        return wxCAL_HITTEST_HEADER;
     }
 
-    int week = (y - m_heightRow) / m_heightRow;
+///////////////////////////////////////////////////////////////////////////////////////////////////////
+    // Header: Days
+    int wday = pos.x / m_widthCol;
+//    if ( y < m_heightRow )
+    if ( y < (m_heightRow + m_rowOffset) )
+    {
+        if ( y > m_rowOffset )
+        {
+            if ( wd )
+            {
+                if ( GetWindowStyle() & wxCAL_MONDAY_FIRST )
+                {
+                    wday = wday == 6 ? 0 : wday + 1;
+                }
+
+                *wd = (wxDateTime::WeekDay)wday;
+            }
+
+            return wxCAL_HITTEST_HEADER;
+        }
+        else
+        {
+            return wxCAL_HITTEST_NOWHERE;
+        }
+    }
+
+//    int week = (y - m_heightRow) / m_heightRow;
+    int week = (y - (m_heightRow + m_rowOffset)) / m_heightRow;
     if ( week >= 6 || wday >= 7 )
     {
         return wxCAL_HITTEST_NOWHERE;
@@ -839,7 +1452,15 @@ wxCalendarHitTestResult wxCalendarCtrl::HitTest(const wxPoint& pos,
         if ( date )
             *date = dt;
 
-        return wxCAL_HITTEST_DAY;
+        if ( dt.GetMonth() == m_date.GetMonth() )
+        {
+
+            return wxCAL_HITTEST_DAY;
+        }
+        else
+        {
+            return wxCAL_HITTEST_SURROUNDING_WEEK;
+        }
     }
     else
     {
@@ -861,9 +1482,10 @@ void wxCalendarCtrl::OnMonthChange(wxCommandEvent& event)
         tm.mday = wxDateTime::GetNumberOfDays(mon, tm.year);
     }
 
-    SetDate(wxDateTime(tm.mday, mon, tm.year));
+    wxDateTime target = wxDateTime(tm.mday, mon, tm.year);
 
-    GenerateEvents(wxEVT_CALENDAR_MONTH_CHANGED, wxEVT_CALENDAR_SEL_CHANGED);
+    ChangeMonth(&target);
+    SetDateAndNotify(target);
 }
 
 void wxCalendarCtrl::OnYearChange(wxSpinEvent& event)
@@ -876,9 +1498,19 @@ void wxCalendarCtrl::OnYearChange(wxSpinEvent& event)
         tm.mday = wxDateTime::GetNumberOfDays(tm.mon, year);
     }
 
-    SetDate(wxDateTime(tm.mday, tm.mon, year));
+    wxDateTime target = wxDateTime(tm.mday, tm.mon, year);
+
+    if ( ChangeYear(&target) )
+    {
+        SetDateAndNotify(target);
+    }
+    else
+    {
+        // In this case we don't want to change the date. That would put us
+        // inside the same year but a strange number of months forward/back..
+        m_spinYear->SetValue(target.GetYear());
+    }
 
-    GenerateEvents(wxEVT_CALENDAR_YEAR_CHANGED, wxEVT_CALENDAR_SEL_CHANGED);
 }
 
 // ----------------------------------------------------------------------------
@@ -887,40 +1519,67 @@ void wxCalendarCtrl::OnYearChange(wxSpinEvent& event)
 
 void wxCalendarCtrl::OnChar(wxKeyEvent& event)
 {
+    wxDateTime target;
     switch ( event.KeyCode() )
     {
         case _T('+'):
         case WXK_ADD:
-            SetDateAndNotify(m_date + wxDateSpan::Year());
+            target = m_date + wxDateSpan::Year();
+            if ( ChangeYear(&target) )
+            {
+                SetDateAndNotify(target);
+            }
             break;
 
         case _T('-'):
         case WXK_SUBTRACT:
-            SetDateAndNotify(m_date - wxDateSpan::Year());
+            target = m_date - wxDateSpan::Year();
+            if ( ChangeYear(&target) )
+            {
+                SetDateAndNotify(target);
+            }
             break;
 
         case WXK_PRIOR:
-            SetDateAndNotify(m_date - wxDateSpan::Month());
+            target = m_date - wxDateSpan::Month();
+            ChangeMonth(&target);
+            SetDateAndNotify(target); // always
             break;
 
         case WXK_NEXT:
-            SetDateAndNotify(m_date + wxDateSpan::Month());
+            target = m_date + wxDateSpan::Month();
+            ChangeMonth(&target);
+            SetDateAndNotify(target); // always
             break;
 
         case WXK_RIGHT:
             if ( event.ControlDown() )
-                SetDateAndNotify(wxDateTime(m_date).SetToNextWeekDay(
+            {
+                target = wxDateTime(m_date).SetToNextWeekDay(
                                  GetWindowStyle() & wxCAL_MONDAY_FIRST
-                                 ? wxDateTime::Sun : wxDateTime::Sat));
+                                 ? wxDateTime::Sun : wxDateTime::Sat);
+                if ( !IsDateInRange(target) )
+                {
+                    target = GetUpperDateLimit();
+                }
+                SetDateAndNotify(target);
+            }
             else
                 SetDateAndNotify(m_date + wxDateSpan::Day());
             break;
 
         case WXK_LEFT:
             if ( event.ControlDown() )
-                SetDateAndNotify(wxDateTime(m_date).SetToPrevWeekDay(
+            {
+                target = wxDateTime(m_date).SetToPrevWeekDay(
                                  GetWindowStyle() & wxCAL_MONDAY_FIRST
-                                 ? wxDateTime::Mon : wxDateTime::Sun));
+                                 ? wxDateTime::Mon : wxDateTime::Sun);
+                if ( !IsDateInRange(target) )
+                {
+                    target = GetLowerDateLimit();
+                }
+                SetDateAndNotify(target);
+            }
             else
                 SetDateAndNotify(m_date - wxDateSpan::Day());
             break;
@@ -957,13 +1616,6 @@ void wxCalendarCtrl::OnChar(wxKeyEvent& event)
 // holidays handling
 // ----------------------------------------------------------------------------
 
-void wxCalendarCtrl::OnCalMonthChange(wxCalendarEvent& event)
-{
-    SetHolidayAttrs();
-
-    event.Skip();
-}
-
 void wxCalendarCtrl::EnableHolidayDisplay(bool display)
 {
     long style = GetWindowStyle();
@@ -1043,4 +1695,8 @@ wxCalendarEvent::wxCalendarEvent(wxCalendarCtrl *cal, wxEventType type)
                : wxCommandEvent(type, cal->GetId())
 {
     m_date = cal->GetDate();
+    SetEventObject(cal);
 }
+
+#endif // wxUSE_CALENDARCTRL
+