// headers
// ----------------------------------------------------------------------------
-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
#pragma implementation "univlistbox.h"
#endif
BEGIN_EVENT_TABLE(wxListBox, wxListBoxBase)
EVT_SIZE(wxListBox::OnSize)
-
- EVT_IDLE(wxListBox::OnIdle)
END_EVENT_TABLE()
// ----------------------------------------------------------------------------
m_maxWidth = 0;
m_scrollRangeY = 0;
m_maxWidthItem = -1;
+ m_strings = NULL;
// no items hence no current item
m_current = -1;
m_showScrollbarY = FALSE;
}
+wxListBox::wxListBox(wxWindow *parent,
+ wxWindowID id,
+ const wxPoint &pos,
+ const wxSize &size,
+ const wxArrayString& choices,
+ long style,
+ const wxValidator& validator,
+ const wxString &name)
+{
+ Init();
+
+ Create(parent, id, pos, size, choices, style, validator, name);
+}
+
+bool wxListBox::Create(wxWindow *parent,
+ wxWindowID id,
+ const wxPoint &pos,
+ const wxSize &size,
+ const wxArrayString& choices,
+ long style,
+ const wxValidator& validator,
+ const wxString &name)
+{
+ wxCArrayString chs(choices);
+
+ return Create(parent, id, pos, size, chs.GetCount(), chs.GetStrings(),
+ style, validator, name);
+}
+
bool wxListBox::Create(wxWindow *parent,
wxWindowID id,
const wxPoint &pos,
if ( !(style & (wxLB_MULTIPLE | wxLB_EXTENDED)) )
style |= wxLB_SINGLE;
- if ( !wxControl::Create(parent, id, pos, size, style, wxDefaultValidator, name) )
+#if wxUSE_TWO_WINDOWS
+ style |= wxVSCROLL|wxHSCROLL;
+ if ((style & wxBORDER_MASK) == 0)
+ style |= wxBORDER_SUNKEN;
+#endif
+
+ if ( !wxControl::Create(parent, id, pos, size, style,
+ validator, name) )
return FALSE;
SetWindow(this);
if ( style & wxLB_SORT )
- m_strings = wxArrayString(TRUE /* auto sort */);
+ m_stringsSorted = new wxSortedArrayString;
+ else
+ m_strings = new wxArrayString;
Set(n, choices);
wxListBox::~wxListBox()
{
+ // call this just to free the client data -- and avoid leaking memory
+ DoClear();
+
+ if ( IsSorted() )
+ delete m_stringsSorted;
+ else
+ delete m_strings;
+
+ m_strings = NULL;
}
// ----------------------------------------------------------------------------
int wxListBox::DoAppend(const wxString& item)
{
- size_t index = m_strings.Add(item);
+ size_t index;
+
+ if ( IsSorted() )
+ {
+ index = m_stringsSorted->Add(item);
+ }
+ else
+ {
+ index = m_strings->GetCount();
+ m_strings->Add(item);
+ }
+
m_itemsClientData.Insert(NULL, index);
m_updateScrollbarY = TRUE;
size_t count = items.GetCount();
for ( size_t n = 0; n < count; n++ )
{
- m_strings.Insert(items[n], pos + n);
+ m_strings->Insert(items[n], pos + n);
m_itemsClientData.Insert(NULL, pos + n);
}
if ( !count )
return;
- m_strings.Alloc(count);
+ m_strings->Alloc(count);
m_itemsClientData.Alloc(count);
for ( size_t n = 0; n < count; n++ )
{
- size_t index = m_strings.Add(items[n]);
+ size_t index;
+
+ if ( IsSorted() )
+ {
+ index = m_stringsSorted->Add(items[n]);
+ }
+ else
+ {
+ index = m_strings->GetCount();
+ m_strings->Add(items[n]);
+ }
+
m_itemsClientData.Insert(clientData ? clientData[n] : NULL, index);
}
// we need to update m_maxWidth as changing the string may cause the
// horz scrollbar [dis]appear
wxCoord width;
- m_strings[n] = s;
+ (*m_strings)[n] = s;
GetTextExtent(s, &width, NULL);
// it might have increased if the new string is long
}
else // no horz scrollbar
{
- m_strings[n] = s;
+ (*m_strings)[n] = s;
}
RefreshItem(n);
void wxListBox::DoClear()
{
- m_strings.Clear();
+ m_strings->Clear();
if ( HasClientObjectData() )
{
void wxListBox::Delete(int n)
{
- wxCHECK_RET( n < GetCount(), _T("invalid index in wxListBox::Delete") );
+ wxCHECK_RET( n >= 0 && n < GetCount(),
+ _T("invalid index in wxListBox::Delete") );
// do it before removing the index as otherwise the last item will not be
// refreshed (as GetCount() will be decremented)
RefreshFromItemToEnd(n);
- m_strings.RemoveAt(n);
+ m_strings->RemoveAt(n);
if ( HasClientObjectData() )
{
}
}
-void wxListBox::OnIdle(wxIdleEvent& event)
+void wxListBox::OnInternalIdle()
{
if ( m_updateScrollbarY || m_updateScrollbarX )
{
m_updateCount = 0;
}
-
- event.Skip();
+ wxListBoxBase::OnInternalIdle();
}
// ----------------------------------------------------------------------------
wxCoord lineHeight = GetLineHeight();
size_t itemFirst = yTop / lineHeight,
itemLast = (yBottom + lineHeight - 1) / lineHeight,
- itemMax = m_strings.GetCount();
+ itemMax = m_strings->GetCount();
if ( itemFirst >= itemMax )
return;
{
wxListBox *self = wxConstCast(this, wxListBox);
wxCoord width;
- size_t count = m_strings.GetCount();
+ size_t count = m_strings->GetCount();
for ( size_t n = 0; n < count; n++ )
{
- GetTextExtent(m_strings[n], &width, NULL);
+ GetTextExtent((*m_strings)[n], &width, NULL);
if ( width > m_maxWidth )
{
self->m_maxWidth = width;
height = ((height - hBorders + hLine - 1) / hLine)*hLine + hBorders;
}
- wxListBoxBase::DoSetSize(x, y, width, height);
+ wxListBoxBase::DoSetSize(x, y, width, height, sizeFlags);
}
wxSize wxListBox::DoGetBestClientSize() const
wxCoord width = 0,
height = 0;
- size_t count = m_strings.GetCount();
+ size_t count = m_strings->GetCount();
for ( size_t n = 0; n < count; n++ )
{
wxCoord w,h;
- GetTextExtent(m_strings[n], &w, &h);
+ GetTextExtent((*m_strings)[n], &w, &h);
if ( w > width )
width = w;
bool wxListBox::SendEvent(wxEventType type, int item)
{
- // don't generate select events while the mouse is captured, we will only
- // send them once it is released
- if ( (type == wxEVT_COMMAND_LISTBOX_SELECTED) && (GetCapture() == this) )
- return FALSE;
-
wxCommandEvent event(type, m_windowId);
event.SetEventObject(this);
// loop over all items in the listbox
for ( int item = first; item != last; item < count - 1 ? item++ : item = 0 )
{
- if ( wxStrnicmp(m_strings[item], prefix, len) == 0 )
+ if ( wxStrnicmp((*m_strings)[item], prefix, len) == 0 )
{
SetCurrentItem(item);
}
}
-void wxListBox::Select(bool sel, int item)
+void wxListBox::DoSelect(int item, bool sel)
{
if ( item != -1 )
{
void wxListBox::SelectAndNotify(int item)
{
- Select(TRUE, item);
+ DoSelect(item);
SendEvent(wxEVT_COMMAND_LISTBOX_SELECTED);
}
if ( item != -1 )
{
- Select(TRUE, item);
+ DoSelect(item);
SendEvent(wxEVT_COMMAND_LISTBOX_DOUBLECLICKED);
}
// input handling
// ----------------------------------------------------------------------------
+/*
+ The numArg here is the listbox item index while the strArg is used
+ differently for the different actions:
+
+ a) for wxACTION_LISTBOX_FIND it has the natural meaning: this is the string
+ to find
+
+ b) for wxACTION_LISTBOX_SELECT and wxACTION_LISTBOX_EXTENDSEL it is used
+ to decide if the listbox should send the notification event (it is empty)
+ or not (it is not): this allows us to reuse the same action for when the
+ user is dragging the mouse when it has been released although in the
+ first case no notification is sent while in the second it is sent.
+ */
bool wxListBox::PerformAction(const wxControlAction& action,
long numArg,
const wxString& strArg)
int item = (int)numArg;
if ( action == wxACTION_LISTBOX_SETFOCUS )
+ {
SetCurrentItem(item);
+ }
else if ( action == wxACTION_LISTBOX_ACTIVATE )
+ {
Activate(item);
+ }
else if ( action == wxACTION_LISTBOX_TOGGLE )
{
if ( item == -1 )
item = m_current;
if ( IsSelected(item) )
- Unselect(item);
+ DoUnselect(item);
else
SelectAndNotify(item);
}
else if ( action == wxACTION_LISTBOX_SELECT )
{
DeselectAll(item);
- SelectAndNotify(item);
+
+ if ( strArg.empty() )
+ SelectAndNotify(item);
+ else
+ DoSelect(item);
}
else if ( action == wxACTION_LISTBOX_SELECTADD )
- Select(TRUE, item);
+ DoSelect(item);
else if ( action == wxACTION_LISTBOX_UNSELECT )
- Select(FALSE, item);
+ DoUnselect(item);
else if ( action == wxACTION_LISTBOX_MOVEDOWN )
ChangeCurrent(1);
else if ( action == wxACTION_LISTBOX_MOVEUP )
switch ( keycode )
{
// movement
- case WXK_UP: action = wxACTION_LISTBOX_MOVEUP; break;
- case WXK_DOWN: action = wxACTION_LISTBOX_MOVEDOWN; break;
+ case WXK_UP:
+ action = wxACTION_LISTBOX_MOVEUP;
+ break;
+
+ case WXK_DOWN:
+ action = wxACTION_LISTBOX_MOVEDOWN;
+ break;
+
case WXK_PAGEUP:
- case WXK_PRIOR: action = wxACTION_LISTBOX_PAGEUP; break;
+
+ case WXK_PRIOR:
+ action = wxACTION_LISTBOX_PAGEUP;
+ break;
+
case WXK_PAGEDOWN:
- case WXK_NEXT: action = wxACTION_LISTBOX_PAGEDOWN; break;
- case WXK_HOME: action = wxACTION_LISTBOX_START; break;
- case WXK_END: action = wxACTION_LISTBOX_END; break;
+
+ case WXK_NEXT:
+ action = wxACTION_LISTBOX_PAGEDOWN;
+ break;
+
+ case WXK_HOME:
+ action = wxACTION_LISTBOX_START;
+ break;
+
+ case WXK_END:
+ action = wxACTION_LISTBOX_END;
+ break;
// selection
case WXK_SPACE:
winCapture->ReleaseMouse();
m_btnCapture = 0;
- // generate the last event to triiger sending the selection event
action = m_actionMouse;
}
//else: the mouse wasn't presed over the listbox, only released here
if ( IsValidIndex(lbox, item) )
{
- lbox->PerformAction(m_actionMouse, item);
+ // pass something into strArg to tell the listbox that it shouldn't
+ // send the notification message: see PerformAction() above
+ lbox->PerformAction(m_actionMouse, item, _T("no"));
}
// else: don't pass invalid index to the listbox
}