Opened 13 years ago

Closed 11 years ago

#534 closed bug (fixed)

[BPopUpMenu] checker glitches

Reported by: diver Owned by: stippi
Priority: normal Milestone: R1
Component: - General Version:
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

screenshot coming...

Attachments (1)

Menu.png (13.4 KB) - added by diver 13 years ago.
R5 Menu pref with shifted checkers

Download all attachments as: .zip

Change History (8)

Changed 13 years ago by diver

Attachment: Menu.png added

R5 Menu pref with shifted checkers

comment:1 Changed 13 years ago by jackburton

Is there any other app which shows this problem ? Could just be that R5 menu preflet expect checkers to be a different size (in other words, it's broken).

comment:2 Changed 13 years ago by korli

Running R5 Menu pref is out of Haiku scope.

comment:3 Changed 13 years ago by jackburton

Exactly what I was thinking, that's why I asked if there were any other app :)

comment:4 Changed 13 years ago by diver

There are problems with checkers with many apps if font set to 14 or higher.

comment:5 Changed 13 years ago by korli

Reproduced with Backgrounds pref popup menus. I think BMenuItem::_DrawMarkSymbol() should take into account the left value of BMenu::GetItemMargins(), along with the BMenuItem height.

comment:6 Changed 13 years ago by jackburton

... or the other way round. I mean BMenu could take into account the font size when calculating the padding for MenuItems.

comment:7 Changed 11 years ago by stippi

Platform: All
Resolution: fixed
Status: newclosed

I've fixed that recently, although the BMenu could take care of setting the margins big enough on BMenuField menus.

Note: See TracTickets for help on using tickets.