#include "wx/log.h"
#include "wx/event.h"
#include "wx/window.h"
-#endif //WX_PRECOMP
-
-#include "wx/containr.h"
-
-#ifdef __WXMAC__
#include "wx/scrolbar.h"
-#endif
-
-#ifdef __WXMSW__
#include "wx/radiobut.h"
-#endif
+ #include "wx/containr.h"
+#endif //WX_PRECOMP
+
+// trace mask for focus messages
+#define TRACE_FOCUS _T("focus")
// ============================================================================
// implementation
wxControlContainer::wxControlContainer(wxWindow *winParent)
{
m_winParent = winParent;
-
- m_winLastFocused =
- m_winTmpDefault =
- m_winDefault = NULL;
+ m_winLastFocused = NULL;
m_inSetFocus = false;
}
while ( node )
{
wxWindow *child = node->GetData();
+ node = node->GetNext();
+#ifdef __WXMAC__
+ if ( m_winParent->MacIsWindowScrollbar( child ) )
+ continue;
+ hasRealChildren = true ;
+#endif
if ( child->AcceptsFocus() )
{
return true;
}
-
-#ifdef __WXMAC__
- wxScrollBar *sb = wxDynamicCast( child , wxScrollBar ) ;
- if ( sb == NULL || !m_winParent->MacIsWindowScrollbar( sb ) )
- hasRealChildren = true ;
-#endif
- node = node->GetNext();
}
#ifdef __WXMAC__
if ( win )
{
- wxLogTrace(_T("focus"), _T("Set last focus to %s(%s)"),
+ wxLogTrace(TRACE_FOCUS, _T("Set last focus to %s(%s)"),
win->GetClassInfo()->GetClassName(),
win->GetLabel().c_str());
}
else
{
- wxLogTrace(_T("focus"), _T("No more last focus"));
+ wxLogTrace(TRACE_FOCUS, _T("No more last focus"));
}
}
// no more buttons in group
return NULL;
}
- else
- return prevBtn;
+
+ return prevBtn;
}
wxRadioButton* wxGetNextButtonInGroup(wxRadioButton *btn)
// no more buttons or the first button of the next group
return NULL;
}
- else
- return nextBtn;
+
+ return nextBtn;
}
wxRadioButton* wxGetFirstButtonInGroup(wxRadioButton *btn)
}
}
+wxRadioButton* wxGetLastButtonInGroup(wxRadioButton *btn)
+{
+ while (true)
+ {
+ wxRadioButton* nextBtn = wxGetNextButtonInGroup(btn);
+ if (!nextBtn)
+ return btn;
+
+ btn = nextBtn;
+ }
+}
+
wxRadioButton* wxGetSelectedButtonInGroup(wxRadioButton *btn)
{
// Find currently selected button
const wxWindowList& children = m_winParent->GetChildren();
+ // if we have exactly one notebook-like child window (actually it could be
+ // any window that returns true from its HasMultiplePages()), then
+ // [Shift-]Ctrl-Tab and Ctrl-PageUp/Down keys should iterate over its pages
+ // even if the focus is outside of the control because this is how the
+ // standard MSW properties dialogs behave and we do it under other platforms
+ // as well because it seems like a good idea -- but we can always put this
+ // block inside "#ifdef __WXMSW__" if it's not suitable there
+ if ( event.IsWindowChange() && !goingDown )
+ {
+ // check if we have a unique notebook-like child
+ wxWindow *bookctrl = NULL;
+ for ( wxWindowList::const_iterator i = children.begin(),
+ end = children.end();
+ i != end;
+ ++i )
+ {
+ wxWindow * const window = *i;
+ if ( window->HasMultiplePages() )
+ {
+ if ( bookctrl )
+ {
+ // this is the second book-like control already so don't do
+ // anything as we don't know which one should have its page
+ // changed
+ bookctrl = NULL;
+ break;
+ }
+
+ bookctrl = window;
+ }
+ }
+
+ if ( bookctrl && bookctrl->GetEventHandler()->ProcessEvent(event) )
+ return;
+ }
+
// there is not much to do if we don't have children and we're not
// interested in "notebook page change" events here
if ( !children.GetCount() || event.IsWindowChange() )
}
// where are we going?
- bool forward = event.GetDirection();
+ const bool forward = event.GetDirection();
// the node of the children list from which we should start looking for the
// next acceptable child
// start from first or last depending on where we're going
node = forward ? children.GetFirst() : children.GetLast();
-
- // we want to cycle over all nodes
- start_node = wxWindowList::compatibility_iterator();
}
- else
+ else // going up
{
// try to find the child which has the focus currently
// ok, we found the focus - now is it our child?
start_node = children.Find( winFocus );
}
- else
- {
- start_node = wxWindowList::compatibility_iterator();
- }
if ( !start_node && m_winLastFocused )
{
}
// we want to cycle over all elements passing by NULL
- while ( node != start_node )
+ for ( ;; )
{
+ // don't go into infinite loop
+ if ( start_node && node && node == start_node )
+ break;
+
// Have we come to the last or first item on the panel?
if ( !node )
{
+ if ( !start_node )
+ {
+ // exit now as otherwise we'd loop forever
+ break;
+ }
+
if ( !goingDown )
{
- // Check if our (may be grand) parent is another panel: if this
+ // Check if our (maybe grand) parent is another panel: if this
// is the case, they will know what to do with this navigation
// key and so give them the chance to process it instead of
// looping inside this panel (normally, the focus will go to
wxWindow *child = node->GetData();
#ifdef __WXMSW__
- bool canSelectRadioButton = true;
- if (!event.IsFromTab())
+ if ( event.IsFromTab() )
{
- // If navigating using cursor keys, make sure not to navigate out of a radio button group.
- if (m_winLastFocused && wxIsKindOf(m_winLastFocused, wxRadioButton))
+ if ( wxIsKindOf(child, wxRadioButton) )
{
- if (!wxIsKindOf(child, wxRadioButton))
+ // only radio buttons with either wxRB_GROUP or wxRB_SINGLE
+ // can be tabbed to
+ if ( child->HasFlag(wxRB_GROUP) )
{
- child = forward ?
- wxGetNextButtonInGroup((wxRadioButton*)m_winLastFocused) :
- wxGetPreviousButtonInGroup((wxRadioButton*)m_winLastFocused);
- if (!child)
- {
- event.Skip(false);
- return;
- }
+ // need to tab into the active button within a group
+ wxRadioButton *rb = wxGetSelectedButtonInGroup((wxRadioButton*)child);
+ if ( rb )
+ child = rb;
+ }
+ else if ( !child->HasFlag(wxRB_SINGLE) )
+ {
+ node = forward ? node->GetNext() : node->GetPrevious();
+ continue;
}
}
}
- else
+ else if ( m_winLastFocused &&
+ wxIsKindOf(m_winLastFocused, wxRadioButton) &&
+ !m_winLastFocused->HasFlag(wxRB_SINGLE) )
{
- // If navigating using tabs, skip all but the first radio button in a group.
- if (wxIsKindOf(child, wxRadioButton))
+ // cursor keys don't navigate out of a radio button group so
+ // find the correct radio button to focus
+ if ( forward )
+ {
+ child = wxGetNextButtonInGroup((wxRadioButton*)m_winLastFocused);
+ if ( !child )
+ {
+ // no next button in group, set it to the first button
+ child = wxGetFirstButtonInGroup((wxRadioButton*)m_winLastFocused);
+ }
+ }
+ else
+ {
+ child = wxGetPreviousButtonInGroup((wxRadioButton*)m_winLastFocused);
+ if ( !child )
+ {
+ // no previous button in group, set it to the last button
+ child = wxGetLastButtonInGroup((wxRadioButton*)m_winLastFocused);
+ }
+ }
+
+ if ( child == m_winLastFocused )
{
- if (wxGetPreviousButtonInGroup((wxRadioButton*)child))
- canSelectRadioButton = false;
+ // must be a group consisting of only one button therefore
+ // no need to send a navigation event
+ event.Skip(false);
+ return;
}
}
-#else
- static bool canSelectRadioButton = true;
-#endif
+#endif // __WXMSW__
- if ( child->AcceptsFocusFromKeyboard() && canSelectRadioButton )
+ if ( child->AcceptsFocusFromKeyboard() )
{
// if we're setting the focus to a child panel we should prevent it
// from giving it to the child which had the focus the last time
// direction we're coming
event.SetEventObject(m_winParent);
-#if defined(__WXMSW__)
- // we need to hop to the next activated
- // radio button, not just the next radio
- // button under MSW
- if (wxIsKindOf(child, wxRadioButton) && event.IsFromTab())
- {
- wxRadioButton *rb = wxGetSelectedButtonInGroup((wxRadioButton*)child);
- if (rb)
- child = rb;
- }
-#endif // __WXMSW__
-
// disable propagation for this call as otherwise the event might
// bounce back to us.
wxPropagationDisabler disableProp(event);
{
if ( child == m_winLastFocused )
m_winLastFocused = NULL;
-
- if ( child == m_winDefault )
- m_winDefault = NULL;
-
- if ( child == m_winTmpDefault )
- m_winTmpDefault = NULL;
}
// ----------------------------------------------------------------------------
bool wxControlContainer::DoSetFocus()
{
- wxLogTrace(_T("focus"), _T("SetFocus on wxPanel 0x%08lx."),
- (unsigned long)m_winParent->GetHandle());
+ wxLogTrace(TRACE_FOCUS, _T("SetFocus on wxPanel 0x%p."),
+ m_winParent->GetHandle());
if (m_inSetFocus)
return true;
void wxControlContainer::HandleOnFocus(wxFocusEvent& event)
{
- wxLogTrace(_T("focus"), _T("OnFocus on wxPanel 0x%08lx, name: %s"),
- (unsigned long)m_winParent->GetHandle(),
+ wxLogTrace(TRACE_FOCUS, _T("OnFocus on wxPanel 0x%p, name: %s"),
+ m_winParent->GetHandle(),
m_winParent->GetName().c_str() );
DoSetFocus();
// It might happen that the window got reparented
if ( (*childLastFocused)->GetParent() == win )
{
- wxLogTrace(_T("focus"),
- _T("SetFocusToChild() => last child (0x%08lx)."),
- (unsigned long)(*childLastFocused)->GetHandle());
+ wxLogTrace(TRACE_FOCUS,
+ _T("SetFocusToChild() => last child (0x%p)."),
+ (*childLastFocused)->GetHandle());
// not SetFocusFromKbd(): we're restoring focus back to the old
// window and not setting it as the result of a kbd action
while ( node )
{
wxWindow *child = node->GetData();
+ node = node->GetNext();
+#ifdef __WXMAC__
+ if ( child->GetParent()->MacIsWindowScrollbar( child ) )
+ continue;
+#endif
+
if ( child->AcceptsFocusFromKeyboard() && !child->IsTopLevel() )
{
#ifdef __WXMSW__
}
#endif
- wxLogTrace(_T("focus"),
- _T("SetFocusToChild() => first child (0x%08lx)."),
- (unsigned long)child->GetHandle());
+ wxLogTrace(TRACE_FOCUS,
+ _T("SetFocusToChild() => first child (0x%p)."),
+ child->GetHandle());
*childLastFocused = child;
child->SetFocusFromKbd();
return true;
}
-
- node = node->GetNext();
}
return false;
}
-