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

BMenuItem draws the various elements in a more correct position, i.e. it doesn't draw the shortcut char over the submenu arrow. Fixes bug 206

Browse Source 
git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@16504 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
Stefano Ceccherini 2006-02-25 21:02:25 +00:00
parent 5d5077a088
commit 8152e71214
1 changed files with 17 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -708,6 +708,18 @@ BMenuItem::DrawMarkSymbol()
void
BMenuItem::DrawShortcutSymbol()
{
// TODO: Review this
BPoint where(ContentLocation() + BPoint(fBounds.Width() - 32.0f, fBounds.Height() - 4.0f));
if (fSubmenu)
where.x -= 12;
// TODO: If the shortcut is one of B_DOWN_ARROW, B_UP_ARROW, B_ENTER, etc.
// we can't just use DrawString(), as those aren't valid ascii/UTF8 charachters.
// In that case we need to build a BBitmap for the given shortcut and draw it using DrawBitmap()
fSuper->DrawChar(fShortcutChar, where);
where -= BPoint(20, 10);
if (fModifiers & B_COMMAND_KEY) {
key_map *keys;
char *chars;
@ -718,9 +730,10 @@ BMenuItem::DrawShortcutSymbol()
control.SetBits(kAltBits, kAltLength, 0, B_COLOR_8_BIT);
else
control.SetBits(kCtrlBits, kCtrlLength, 0, B_COLOR_8_BIT);
fSuper->DrawBitmap(&control, rect, rect.OffsetByCopy(ContentLocation() +
BPoint(fBounds.Width() - 52.0f, fBounds.Height() - 4.0f - rect.Height())));
fSuper->DrawBitmap(&control, where);
where.x -= rect.Width();
free(chars);
free(keys);
}
@ -728,15 +741,8 @@ BMenuItem::DrawShortcutSymbol()
BRect rect(0,0,21,10);
BBitmap shift(rect, B_COLOR_8_BIT);
shift.SetBits(kShiftBits, kShiftLength, 0, B_COLOR_8_BIT);
fSuper->DrawBitmap(&shift, rect, rect.OffsetByCopy(ContentLocation() +
BPoint(fBounds.Width() - 74.0f, fBounds.Height() - 4.0f - rect.Height())));
fSuper->DrawBitmap(&shift, where - BPoint(6, 0));
}
// TODO: If the shortcut is one of B_DOWN_ARROW, B_UP_ARROW, B_ENTER, etc.
// we can't just use DrawString(), as those aren't valid ascii/UTF8 charachters.
// In that case we need to build a BBitmap for the given shortcut and draw it using DrawBitmap()
fSuper->DrawChar(fShortcutChar, ContentLocation() +
BPoint(fBounds.Width() - 32.0f, fBounds.Height() - 4.0f));
}