Aren/haiku
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

This should fix some problems I experienced later. At least, I can't reproduce them anymore

Browse Source 
git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@16920 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
Stefano Ceccherini 2006-03-29 10:42:54 +00:00
parent d70f0203a2
commit 793fde40eb
2 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/kits/interface/Menu.cpp
View File

@ -1266,15 +1266,16 @@ BMenu::RemoveItems(int32 index, int32 count, BMenuItem *item, bool deleteItems)
bool invalidateLayout = false;
bool locked = LockLooper();
BWindow *window = Window();
// The plan is simple: If we're given a BMenuItem directly, we use it
// and ignore index and count. Otherwise, we use them instead.
if (item != NULL) {
if (fItems.RemoveItem(item)) {
if (item == fSelected)
if (item == fSelected && window != NULL)
SelectItem(NULL);
item->SetSuper(NULL);
item->Uninstall();
item->SetSuper(NULL);
if (deleteItems)
delete item;
success = invalidateLayout = true;
@ -1290,10 +1291,10 @@ BMenu::RemoveItems(int32 index, int32 count, BMenuItem *item, bool deleteItems)
item = static_cast<BMenuItem*>(fItems.ItemAt(i));
if (item != NULL) {
if (fItems.RemoveItem(item)) {
if (item == fSelected)
if (item == fSelected && window != NULL)
SelectItem(NULL);
item->SetSuper(NULL);
item->Uninstall();
item->SetSuper(NULL);
if (deleteItems)
delete item;
success = true;
@ -1307,8 +1308,10 @@ BMenu::RemoveItems(int32 index, int32 count, BMenuItem *item, bool deleteItems)
}
}
if (invalidateLayout && locked && fResizeToFit)
InvalidateLayout();
if (invalidateLayout && locked && fResizeToFit) {
LayoutItems(0);
Invalidate();
}
if (locked)
UnlockLooper();

2
src/kits/interface/MenuItem.cpp
View File

@ -630,7 +630,7 @@ BMenuItem::Uninstall()
if (Target() == fWindow)
SetTarget(BMessenger());
if (fShortcutChar != 0 && fModifiers != 0 && fWindow != NULL)
if (fShortcutChar != 0 && (fModifiers & B_COMMAND_KEY) && fWindow != NULL)
fWindow->RemoveShortcut(fShortcutChar, fModifiers);
fWindow = NULL;