#if wxUSE_MENUS
-#include <ctype.h>
-
#ifndef WX_PRECOMP
#include "wx/intl.h"
#include "wx/log.h"
#include "wx/menu.h"
#endif
+#include "wx/stockitem.h"
+
// ----------------------------------------------------------------------------
// template lists
// ----------------------------------------------------------------------------
// ============================================================================
// ----------------------------------------------------------------------------
-// wxMenuItem
+// wxMenuItemBase
// ----------------------------------------------------------------------------
wxMenuItemBase::wxMenuItemBase(wxMenu *parentMenu,
const wxString& help,
wxItemKind kind,
wxMenu *subMenu)
- : m_text(text),
- m_help(help)
{
wxASSERT_MSG( parentMenu != NULL, wxT("menuitem should have a menu") );
m_id = id;
m_kind = kind;
if (m_id == wxID_ANY)
- m_id = wxNewId();
+ m_id = wxWindow::NewControlId();
if (m_id == wxID_SEPARATOR)
m_kind = wxITEM_SEPARATOR;
+
+ SetItemLabel(text);
+ SetHelp(help);
}
wxMenuItemBase::~wxMenuItemBase()
#if wxUSE_ACCEL
-static const struct wxKeyName
-{
- wxKeyCode code;
- const wxChar *name;
-} wxKeyNames[] =
-{
- { WXK_DELETE, wxTRANSLATE("DEL") },
- { WXK_DELETE, wxTRANSLATE("DELETE") },
- { WXK_BACK, wxTRANSLATE("BACK") },
- { WXK_INSERT, wxTRANSLATE("INS") },
- { WXK_INSERT, wxTRANSLATE("INSERT") },
- { WXK_RETURN, wxTRANSLATE("ENTER") },
- { WXK_RETURN, wxTRANSLATE("RETURN") },
- { WXK_PAGEUP, wxTRANSLATE("PGUP") },
- { WXK_PAGEDOWN, wxTRANSLATE("PGDN") },
- { WXK_LEFT, wxTRANSLATE("LEFT") },
- { WXK_RIGHT, wxTRANSLATE("RIGHT") },
- { WXK_UP, wxTRANSLATE("UP") },
- { WXK_DOWN, wxTRANSLATE("DOWN") },
- { WXK_HOME, wxTRANSLATE("HOME") },
- { WXK_END, wxTRANSLATE("END") },
- { WXK_SPACE, wxTRANSLATE("SPACE") },
- { WXK_TAB, wxTRANSLATE("TAB") },
- { WXK_ESCAPE, wxTRANSLATE("ESC") },
- { WXK_ESCAPE, wxTRANSLATE("ESCAPE") },
- { WXK_CANCEL, wxTRANSLATE("CANCEL") },
- { WXK_CLEAR, wxTRANSLATE("CLEAR") },
- { WXK_MENU, wxTRANSLATE("MENU") },
- { WXK_PAUSE, wxTRANSLATE("PAUSE") },
- { WXK_CAPITAL, wxTRANSLATE("CAPITAL") },
- { WXK_SELECT, wxTRANSLATE("SELECT") },
- { WXK_PRINT, wxTRANSLATE("PRINT") },
- { WXK_EXECUTE, wxTRANSLATE("EXECUTE") },
- { WXK_SNAPSHOT, wxTRANSLATE("SNAPSHOT") },
- { WXK_HELP, wxTRANSLATE("HELP") },
- { WXK_ADD, wxTRANSLATE("ADD") },
- { WXK_SEPARATOR, wxTRANSLATE("SEPARATOR") },
- { WXK_SUBTRACT, wxTRANSLATE("SUBTRACT") },
- { WXK_DECIMAL, wxTRANSLATE("DECIMAL") },
- { WXK_DIVIDE, wxTRANSLATE("DIVIDE") },
- { WXK_NUMLOCK, wxTRANSLATE("NUM_LOCK") },
- { WXK_SCROLL, wxTRANSLATE("SCROLL_LOCK") },
- { WXK_PAGEUP, wxTRANSLATE("PAGEUP") },
- { WXK_PAGEDOWN, wxTRANSLATE("PAGEDOWN") },
- { WXK_NUMPAD_SPACE, wxTRANSLATE("KP_SPACE") },
- { WXK_NUMPAD_TAB, wxTRANSLATE("KP_TAB") },
- { WXK_NUMPAD_ENTER, wxTRANSLATE("KP_ENTER") },
- { WXK_NUMPAD_HOME, wxTRANSLATE("KP_HOME") },
- { WXK_NUMPAD_LEFT, wxTRANSLATE("KP_LEFT") },
- { WXK_NUMPAD_UP, wxTRANSLATE("KP_UP") },
- { WXK_NUMPAD_RIGHT, wxTRANSLATE("KP_RIGHT") },
- { WXK_NUMPAD_DOWN, wxTRANSLATE("KP_DOWN") },
- { WXK_NUMPAD_PAGEUP, wxTRANSLATE("KP_PRIOR") },
- { WXK_NUMPAD_PAGEUP, wxTRANSLATE("KP_PAGEUP") },
- { WXK_NUMPAD_PAGEDOWN, wxTRANSLATE("KP_NEXT") },
- { WXK_NUMPAD_PAGEDOWN, wxTRANSLATE("KP_PAGEDOWN") },
- { WXK_NUMPAD_END, wxTRANSLATE("KP_END") },
- { WXK_NUMPAD_BEGIN, wxTRANSLATE("KP_BEGIN") },
- { WXK_NUMPAD_INSERT, wxTRANSLATE("KP_INSERT") },
- { WXK_NUMPAD_DELETE, wxTRANSLATE("KP_DELETE") },
- { WXK_NUMPAD_EQUAL, wxTRANSLATE("KP_EQUAL") },
- { WXK_NUMPAD_MULTIPLY, wxTRANSLATE("KP_MULTIPLY") },
- { WXK_NUMPAD_ADD, wxTRANSLATE("KP_ADD") },
- { WXK_NUMPAD_SEPARATOR, wxTRANSLATE("KP_SEPARATOR") },
- { WXK_NUMPAD_SUBTRACT, wxTRANSLATE("KP_SUBTRACT") },
- { WXK_NUMPAD_DECIMAL, wxTRANSLATE("KP_DECIMAL") },
- { WXK_NUMPAD_DIVIDE, wxTRANSLATE("KP_DIVIDE") },
- { WXK_WINDOWS_LEFT, wxTRANSLATE("WINDOWS_LEFT") },
- { WXK_WINDOWS_RIGHT, wxTRANSLATE("WINDOWS_RIGHT") },
- { WXK_WINDOWS_MENU, wxTRANSLATE("WINDOWS_MENU") },
- { WXK_COMMAND, wxTRANSLATE("COMMAND") },
-};
-
-// return true if the 2 strings refer to the same accel
-//
-// as accels can be either translated or not, check for both possibilities and
-// also compare case-insensitively as the key names case doesn't count
-static inline bool CompareAccelString(const wxString& str, const wxChar *accel)
+wxAcceleratorEntry *wxMenuItemBase::GetAccel() const
{
- return str.CmpNoCase(accel) == 0
-#if wxUSE_INTL
- || str.CmpNoCase(wxGetTranslation(accel)) == 0
-#endif
- ;
+ return wxAcceleratorEntry::Create(GetItemLabel());
}
-// return prefixCode+number if the string is of the form "<prefix><number>" and
-// 0 if it isn't
-//
-// first and last parameter specify the valid domain for "number" part
-static int
-IsNumberedAccelKey(const wxString& str,
- const wxChar *prefix,
- wxKeyCode prefixCode,
- unsigned first,
- unsigned last)
+void wxMenuItemBase::SetAccel(wxAcceleratorEntry *accel)
{
- const size_t lenPrefix = wxStrlen(prefix);
- if ( !CompareAccelString(str.Left(lenPrefix), prefix) )
- return 0;
-
- unsigned long num;
- if ( !str.Mid(lenPrefix).ToULong(&num) )
- return 0;
-
- if ( num < first || num > last )
+ wxString text = m_text.BeforeFirst(wxT('\t'));
+ if ( accel )
{
- // this must be a mistake, chances that this is a valid name of another
- // key are vanishingly small
- wxLogDebug(_T("Invalid key string \"%s\""), str.c_str());
- return 0;
+ text += wxT('\t');
+ text += accel->ToString();
}
- return prefixCode + num - first;
+ SetItemLabel(text);
}
-// return wxAcceleratorEntry for the given menu string or NULL if none
-wxAcceleratorEntry *wxGetAccelFromString(const wxString& label)
-{
- // check for accelerators: they are given after '\t'
- int posTab = label.Find(wxT('\t'));
- if ( posTab == wxNOT_FOUND )
- return NULL;
-
- // parse the accelerator string
- int accelFlags = wxACCEL_NORMAL;
- wxString current;
- for ( size_t n = (size_t)posTab + 1; n < label.length(); n++ )
- {
- if ( (label[n] == '+') || (label[n] == '-') )
- {
- if ( CompareAccelString(current, wxTRANSLATE("ctrl")) )
- accelFlags |= wxACCEL_CTRL;
- else if ( CompareAccelString(current, wxTRANSLATE("alt")) )
- accelFlags |= wxACCEL_ALT;
- else if ( CompareAccelString(current, wxTRANSLATE("shift")) )
- accelFlags |= wxACCEL_SHIFT;
- else // not a recognized modifier name
- {
- // we may have "Ctrl-+", for example, but we still want to
- // catch typos like "Crtl-A" so only give the warning if we
- // have something before the current '+' or '-', else take
- // it as a literal symbol
- if ( current.empty() )
- {
- current += label[n];
-
- // skip clearing it below
- continue;
- }
- else
- {
- wxLogDebug(wxT("Unknown accel modifier: '%s'"),
- current.c_str());
- }
- }
+#endif // wxUSE_ACCEL
- current.clear();
- }
- else // not special character
- {
- current += (wxChar) wxTolower(label[n]);
- }
- }
+void wxMenuItemBase::SetItemLabel(const wxString& str)
+{
+ m_text = str;
- int keyCode;
- const size_t len = current.length();
- switch ( len )
+ if ( m_text.empty() && !IsSeparator() )
{
- case 0:
- wxLogDebug(wxT("No accel key found, accel string ignored."));
- return NULL;
-
- case 1:
- // it's just a letter
- keyCode = current[0U];
-
- // if the key is used with any modifiers, make it an uppercase one
- // because Ctrl-A and Ctrl-a are the same; but keep it as is if it's
- // used alone as 'a' and 'A' are different
- if ( accelFlags != wxACCEL_NORMAL )
- keyCode = wxToupper(keyCode);
- break;
-
- default:
- keyCode = IsNumberedAccelKey(current, wxTRANSLATE("F"),
- WXK_F1, 1, 12);
- if ( !keyCode )
- {
- for ( size_t n = 0; n < WXSIZEOF(wxKeyNames); n++ )
- {
- const wxKeyName& kn = wxKeyNames[n];
- if ( CompareAccelString(current, kn.name) )
- {
- keyCode = kn.code;
- break;
- }
- }
- }
-
- if ( !keyCode )
- keyCode = IsNumberedAccelKey(current, wxTRANSLATE("KP_"),
- WXK_NUMPAD0, 0, 9);
- if ( !keyCode )
- keyCode = IsNumberedAccelKey(current, wxTRANSLATE("SPECIAL"),
- WXK_SPECIAL1, 1, 20);
-
- if ( !keyCode )
- {
- wxLogDebug(wxT("Unrecognized accel key '%s', accel string ignored."),
- current.c_str());
- return NULL;
- }
+ wxASSERT_MSG( wxIsStockID(GetId()),
+ wxT("A non-stock menu item with an empty label?") );
+ m_text = wxGetStockLabel(GetId(), wxSTOCK_WITH_ACCELERATOR |
+ wxSTOCK_WITH_MNEMONIC);
}
-
-
- wxASSERT_MSG( keyCode, _T("logic error: should have key code here") );
-
- return new wxAcceleratorEntry(accelFlags, keyCode);
}
-wxAcceleratorEntry *wxMenuItemBase::GetAccel() const
+void wxMenuItemBase::SetHelp(const wxString& str)
{
- return wxGetAccelFromString(GetText());
-}
+ m_help = str;
-void wxMenuItemBase::SetAccel(wxAcceleratorEntry *accel)
-{
- wxString text = m_text.BeforeFirst(wxT('\t'));
- if ( accel )
+ if ( m_help.empty() && !IsSeparator() && wxIsStockID(GetId()) )
{
- text += wxT('\t');
-
- int flags = accel->GetFlags();
- if ( flags & wxACCEL_ALT )
- text += _("Alt-");
- if ( flags & wxACCEL_CTRL )
- text += _("Ctrl-");
- if ( flags & wxACCEL_SHIFT )
- text += _("Shift-");
-
- const int code = accel->GetKeyCode();
-
- if ( wxIsalnum(code) )
- text << (wxChar)code;
- else if ( code >= WXK_F1 && code <= WXK_F12 )
- text << _("F") << code - WXK_F1 + 1;
- else if ( code >= WXK_NUMPAD0 && code <= WXK_NUMPAD9 )
- text << _("KP_") << code - WXK_NUMPAD0;
- else if ( code >= WXK_SPECIAL1 && code <= WXK_SPECIAL20 )
- text << _("SPECIAL") << code - WXK_SPECIAL1 + 1;
- else // check the named keys
- {
- size_t n;
- for ( n = 0; n < WXSIZEOF(wxKeyNames); n++ )
- {
- const wxKeyName& kn = wxKeyNames[n];
- if ( code == kn.code )
- {
- text << wxGetTranslation(kn.name);
- break;
- }
- }
-
- wxASSERT_MSG( n != WXSIZEOF(wxKeyNames),
- wxT("unknown keyboard accelerator code") );
- }
+ // get a stock help string
+ m_help = wxGetStockHelpString(GetId());
}
-
- SetText(text);
}
-#endif // wxUSE_ACCEL
+#if WXWIN_COMPATIBILITY_2_8
+wxString wxMenuItemBase::GetLabelFromText(const wxString& text)
+{
+ return GetLabelText(text);
+}
+#endif
bool wxMenuBase::ms_locked = true;
wxMenuBase::~wxMenuBase()
{
WX_CLEAR_LIST(wxMenuItemList, m_items);
-
- // Actually, in GTK, the submenus have to get deleted first.
}
// ----------------------------------------------------------------------------
// Finds the item id matching the given string, wxNOT_FOUND if not found.
int wxMenuBase::FindItem(const wxString& text) const
{
- wxString label = wxMenuItem::GetLabelFromText(text);
+ wxString label = wxMenuItem::GetLabelText(text);
for ( wxMenuItemList::compatibility_iterator node = m_items.GetFirst();
node;
node = node->GetNext() )
// name just like the ordinary items
if ( !item->IsSeparator() )
{
- if ( item->GetLabel() == label )
+ if ( item->GetItemLabelText() == label )
return item->GetId();
}
}
{
wxEvtHandler *handler = GetEventHandler();
if ( handler )
- processed = handler->ProcessEvent(event);
+ processed = handler->SafelyProcessEvent(event);
}
// Try the window the menu was popped up from (and up through the
wxWindow *win = menu->GetInvokingWindow();
if ( win )
{
- processed = win->GetEventHandler()->ProcessEvent(event);
+ processed = win->HandleWindowEvent(event);
break;
}
wxCHECK_RET( item, wxT("wxMenu::SetLabel: no such item") );
- item->SetText(label);
+ item->SetItemLabel(label);
}
wxString wxMenuBase::GetLabel( int id ) const
wxCHECK_MSG( item, wxEmptyString, wxT("wxMenu::GetLabel: no such item") );
- return item->GetText();
+ return item->GetItemLabel();
}
void wxMenuBase::SetHelpString( int id, const wxString& helpString )
int wxMenuBarBase::FindMenu(const wxString& title) const
{
- wxString label = wxMenuItem::GetLabelFromText(title);
+ wxString label = wxMenuItem::GetLabelText(title);
size_t count = GetMenuCount();
for ( size_t i = 0; i < count; i++ )
{
- wxString title2 = GetLabelTop(i);
+ wxString title2 = GetMenuLabel(i);
if ( (title2 == title) ||
- (wxMenuItem::GetLabelFromText(title2) == label) )
+ (wxMenuItem::GetLabelText(title2) == label) )
{
// found
return (int)i;
int wxMenuBarBase::FindMenuItem(const wxString& menu, const wxString& item) const
{
- wxString label = wxMenuItem::GetLabelFromText(menu);
+ wxString label = wxMenuItem::GetLabelText(menu);
int i = 0;
wxMenuList::compatibility_iterator node;
for ( node = m_menus.GetFirst(); node; node = node->GetNext(), i++ )
{
- if ( label == wxMenuItem::GetLabelFromText(GetLabelTop(i)) )
+ if ( label == wxMenuItem::GetLabelText(GetMenuLabel(i)) )
return node->GetData()->FindItem(item);
}
wxCHECK_RET( item, wxT("wxMenuBar::SetLabel(): no such item") );
- item->SetText(label);
+ item->SetItemLabel(label);
}
wxString wxMenuBarBase::GetLabel(int id) const
wxCHECK_MSG( item, wxEmptyString,
wxT("wxMenuBar::GetLabel(): no such item") );
- return item->GetText();
+ return item->GetItemLabel();
}
void wxMenuBarBase::SetHelpString(int id, const wxString& helpString)
}
}
+#if WXWIN_COMPATIBILITY_2_8
+// get or change the label of the menu at given position
+void wxMenuBarBase::SetLabelTop(size_t pos, const wxString& label)
+{
+ SetMenuLabel(pos, label);
+}
+
+wxString wxMenuBarBase::GetLabelTop(size_t pos) const
+{
+ return GetMenuLabelText(pos);
+}
+#endif
+
#endif // wxUSE_MENUS
projects / wxWidgets.git / blobdiff