///////////////////////////////////////////////////////////////////////////////
-// Name: generic/vlbox.cpp
+// Name: src/generic/vlbox.cpp
// Purpose: implementation of wxVListBox
// Author: Vadim Zeitlin
// Modified by:
#pragma hdrstop
#endif
+#if wxUSE_LISTBOX
+
+#include "wx/vlbox.h"
+
#ifndef WX_PRECOMP
#include "wx/settings.h"
#include "wx/dcclient.h"
+ #include "wx/listbox.h"
#endif //WX_PRECOMP
-#include "wx/vlbox.h"
+#include "wx/dcbuffer.h"
#include "wx/selstore.h"
// ----------------------------------------------------------------------------
// implementation
// ============================================================================
+IMPLEMENT_ABSTRACT_CLASS(wxVListBox, wxVScrolledWindow)
+
// ----------------------------------------------------------------------------
// wxVListBox creation
// ----------------------------------------------------------------------------
void wxVListBox::Init()
{
- m_current = wxNOT_FOUND;
+ m_current =
+ m_anchor = wxNOT_FOUND;
m_selStore = NULL;
+ m_doubleBuffer = NULL;
}
bool wxVListBox::Create(wxWindow *parent,
long style,
const wxString& name)
{
+ style |= wxWANTS_CHARS | wxFULL_REPAINT_ON_RESIZE;
if ( !wxVScrolledWindow::Create(parent, id, pos, size, style, name) )
return false;
- SetBackgroundColour(wxSystemSettings::GetColour(wxSYS_COLOUR_LISTBOX));
-
if ( style & wxLB_MULTIPLE )
m_selStore = new wxSelectionStore;
+ // make sure the native widget has the right colour since we do
+ // transparent drawing by default
+ SetBackgroundColour(GetBackgroundColour());
+ m_colBgSel = wxSystemSettings::GetColour(wxSYS_COLOUR_HIGHLIGHT);
+
+ // flicker-free drawing requires this
+ SetBackgroundStyle(wxBG_STYLE_CUSTOM);
+
return true;
}
wxVListBox::~wxVListBox()
{
+ delete m_doubleBuffer;
delete m_selStore;
}
bool wxVListBox::DoSetCurrent(int current)
{
+ wxASSERT_MSG( current == wxNOT_FOUND ||
+ (current >= 0 && (size_t)current < GetItemCount()),
+ _T("wxVListBox::DoSetCurrent(): invalid item index") );
+
if ( current == m_current )
{
// nothing to do
{
// it is, indeed, only partly visible, so scroll it into view to
// make it entirely visible
- if ( (size_t)m_current == GetLastVisibleLine() )
- {
- ScrollToLine(m_current);
- }
+ while ( (size_t)m_current == GetLastVisibleLine() &&
+ ScrollToLine(GetVisibleBegin()+1) ) ;
// but in any case refresh it as even if it was only partly visible
// before we need to redraw it entirely as its background changed
wxCommandEvent event(wxEVT_COMMAND_LISTBOX_SELECTED, GetId());
event.SetEventObject(this);
- event.m_commandInt = m_current;
+ event.SetInt(m_current);
(void)GetEventHandler()->ProcessEvent(event);
}
void wxVListBox::SetSelection(int selection)
{
- wxASSERT_MSG( !HasMultipleSelection(),
- _T("SetSelection() is invalid with multiselection listbox") );
+ wxCHECK_RET( selection == wxNOT_FOUND ||
+ (selection >= 0 && (size_t)selection < GetItemCount()),
+ _T("wxVListBox::SetSelection(): invalid item index") );
+
+ if ( HasMultipleSelection() )
+ {
+ Select(selection);
+ m_anchor = selection;
+ }
DoSetCurrent(selection);
}
}
// ----------------------------------------------------------------------------
-// wxVListBox painting
+// wxVListBox appearance parameters
// ----------------------------------------------------------------------------
void wxVListBox::SetMargins(const wxPoint& pt)
}
}
+void wxVListBox::SetSelectionBackground(const wxColour& col)
+{
+ m_colBgSel = col;
+}
+
+// ----------------------------------------------------------------------------
+// wxVListBox painting
+// ----------------------------------------------------------------------------
+
wxCoord wxVListBox::OnGetLineHeight(size_t line) const
{
return OnMeasureItem(line) + 2*m_ptMargins.y;
{
}
-void wxVListBox::OnPaint(wxPaintEvent& event)
+void wxVListBox::OnDrawBackground(wxDC& dc, const wxRect& rect, size_t n) const
{
- wxPaintDC dc(this);
+ // we need to render selected and current items differently
+ const bool isSelected = IsSelected(n),
+ isCurrent = IsCurrent(n);
+ if ( isSelected || isCurrent )
+ {
+ if ( isSelected )
+ {
+ dc.SetBrush(wxBrush(m_colBgSel, wxSOLID));
+ }
+ else // !selected
+ {
+ dc.SetBrush(*wxTRANSPARENT_BRUSH);
+ }
+
+ dc.SetPen(*(isCurrent ? wxBLACK_PEN : wxTRANSPARENT_PEN));
+
+ dc.DrawRectangle(rect);
+ }
+ //else: do nothing for the normal items
+}
+
+void wxVListBox::OnPaint(wxPaintEvent& WXUNUSED(event))
+{
+ // If size is larger, recalculate double buffer bitmap
+ wxSize clientSize = GetClientSize();
+
+ if ( !m_doubleBuffer ||
+ clientSize.x > m_doubleBuffer->GetWidth() ||
+ clientSize.y > m_doubleBuffer->GetHeight() )
+ {
+ delete m_doubleBuffer;
+ m_doubleBuffer = new wxBitmap(clientSize.x+25,clientSize.y+25);
+ }
+
+ wxBufferedPaintDC dc(this,*m_doubleBuffer);
// the update rectangle
wxRect rectUpdate = GetUpdateClientRect();
+ // fill it with background colour
+ dc.SetBackground(GetBackgroundColour());
+ dc.Clear();
+
// the bounding rectangle of the current line
wxRect rectLine;
- rectLine.width = GetClientSize().x;
+ rectLine.width = clientSize.x;
// iterate over all visible lines
- const size_t lineMax = GetLastVisibleLine();
- for ( size_t line = GetFirstVisibleLine(); line <= lineMax; line++ )
+ const size_t lineMax = GetVisibleEnd();
+ for ( size_t line = GetFirstVisibleLine(); line < lineMax; line++ )
{
const wxCoord hLine = OnGetLineHeight(line);
// don't allow drawing outside of the lines rectangle
wxDCClipper clip(dc, rectLine);
- // we need to render selected and current items differently
- const bool isSelected = IsSelected(line);
- if ( isSelected || IsCurrent(line) )
- {
- if ( isSelected )
- {
- wxBrush brush(wxSystemSettings::
- GetColour(wxSYS_COLOUR_HIGHLIGHT),
- wxSOLID);
- dc.SetBrush(brush);
- }
- else // !selected
- {
- dc.SetBrush(*wxTRANSPARENT_BRUSH);
- }
-
- dc.SetPen(*(IsCurrent(line) ? wxBLACK_PEN : wxTRANSPARENT_PEN));
-
- dc.DrawRectangle(rectLine);
- }
-
wxRect rect = rectLine;
+ OnDrawBackground(dc, rect, line);
+
OnDrawSeparator(dc, rect, line);
rect.Deflate(m_ptMargins.x, m_ptMargins.y);
// wxVListBox keyboard/mouse handling
// ============================================================================
-void wxVListBox::DoHandleItemClick(int item, bool shiftDown, bool ctrlDown)
+void wxVListBox::DoHandleItemClick(int item, int flags)
{
// has anything worth telling the client code about happened?
bool notify = false;
// NB: the keyboard interface we implement here corresponds to
// wxLB_EXTENDED rather than wxLB_MULTIPLE but this one makes more
// sense IMHO
- if ( shiftDown )
+ if ( flags & ItemClick_Shift )
{
if ( m_current != wxNOT_FOUND )
{
+ if ( m_anchor == wxNOT_FOUND )
+ m_anchor = m_current;
+
select = false;
- // only the range from old m_current to new m_current must be
- // selected
+ // only the range from the selection anchor to new m_current
+ // must be selected
if ( DeselectAll() )
notify = true;
- if ( SelectRange(m_current, item) )
+ if ( SelectRange(m_anchor, item) )
notify = true;
}
//else: treat it as ordinary click/keypress
}
- else if ( ctrlDown )
+ else // Shift not pressed
{
- select = false;
+ m_anchor = item;
- Toggle(item);
+ if ( flags & ItemClick_Ctrl )
+ {
+ select = false;
- // the status of the item has definitely changed
- notify = true;
+ if ( !(flags & ItemClick_Kbd) )
+ {
+ Toggle(item);
+
+ // the status of the item has definitely changed
+ notify = true;
+ }
+ //else: Ctrl-arrow pressed, don't change selection
+ }
+ //else: behave as in single selection case
}
- //else: behave as in single selection case
if ( select )
{
void wxVListBox::OnKeyDown(wxKeyEvent& event)
{
- int current = 0; // just to silent the stupid compiler warnings
+ // flags for DoHandleItemClick()
+ int flags = ItemClick_Kbd;
+
+ int current;
switch ( event.GetKeyCode() )
{
case WXK_HOME:
break;
case WXK_PAGEDOWN:
- case WXK_NEXT:
PageDown();
current = GetFirstVisibleLine();
break;
case WXK_PAGEUP:
- case WXK_PRIOR:
if ( m_current == (int)GetFirstVisibleLine() )
{
PageUp();
current = GetFirstVisibleLine();
break;
+ case WXK_SPACE:
+ // hack: pressing space should work like a mouse click rather than
+ // like a keyboard arrow press, so trick DoHandleItemClick() in
+ // thinking we were clicked
+ flags &= ~ItemClick_Kbd;
+ current = m_current;
+ break;
+
+#ifdef __WXMSW__
+ case WXK_TAB:
+ // Since we are using wxWANTS_CHARS we need to send navigation
+ // events for the tabs on MSW
+ {
+ wxNavigationKeyEvent ne;
+ ne.SetDirection(!event.ShiftDown());
+ ne.SetCurrentFocus(this);
+ ne.SetEventObject(this);
+ GetParent()->GetEventHandler()->ProcessEvent(ne);
+ }
+ // fall through to default
+#endif
default:
event.Skip();
+ current = 0; // just to silent the stupid compiler warnings
+ wxUnusedVar(current);
return;
}
- DoHandleItemClick(current, event.ShiftDown(), event.ControlDown());
+ if ( event.ShiftDown() )
+ flags |= ItemClick_Shift;
+ if ( event.ControlDown() )
+ flags |= ItemClick_Ctrl;
+
+ DoHandleItemClick(current, flags);
}
// ----------------------------------------------------------------------------
void wxVListBox::OnLeftDown(wxMouseEvent& event)
{
+ SetFocus();
+
int item = HitTest(event.GetPosition());
- DoHandleItemClick(item, event.ShiftDown(),
+ if ( item != wxNOT_FOUND )
+ {
+ int flags = 0;
+ if ( event.ShiftDown() )
+ flags |= ItemClick_Shift;
+
+ // under Mac Apple-click is used in the same way as Ctrl-click
+ // elsewhere
#ifdef __WXMAC__
- event.MetaDown()
+ if ( event.MetaDown() )
#else
- event.ControlDown()
+ if ( event.ControlDown() )
#endif
- );
+ flags |= ItemClick_Ctrl;
+
+ DoHandleItemClick(item, flags);
+ }
}
-void wxVListBox::OnLeftDClick(wxMouseEvent& event)
+void wxVListBox::OnLeftDClick(wxMouseEvent& eventMouse)
{
- int item = HitTest(event.GetPosition());
+ int item = HitTest(eventMouse.GetPosition());
if ( item != wxNOT_FOUND )
{
- wxCommandEvent event(wxEVT_COMMAND_LISTBOX_DOUBLECLICKED, GetId());
- event.SetEventObject(this);
- event.m_commandInt = item;
- (void)GetEventHandler()->ProcessEvent(event);
+ // if item double-clicked was not yet selected, then treat
+ // this event as a left-click instead
+ if ( item == m_current )
+ {
+ wxCommandEvent event(wxEVT_COMMAND_LISTBOX_DOUBLECLICKED, GetId());
+ event.SetEventObject(this);
+ event.SetInt(item);
+
+ (void)GetEventHandler()->ProcessEvent(event);
+ }
+ else
+ {
+ OnLeftDown(eventMouse);
+ }
+
}
}
+
+// ----------------------------------------------------------------------------
+// use the same default attributes as wxListBox
+// ----------------------------------------------------------------------------
+
+//static
+wxVisualAttributes
+wxVListBox::GetClassDefaultAttributes(wxWindowVariant variant)
+{
+ return wxListBox::GetClassDefaultAttributes(variant);
+}
+
+#endif