Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#902 closed bug (fixed)

BMenu expanding off screen limits

Reported by: sil2100 Owned by: axeld
Priority: normal Milestone: R1
Component: Kits/Interface Kit Version: R1/pre-alpha1
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

As for hrev19045:

The BMenu window, when too big to fit to the right side of its parent, it's moved to the left side. The problem is that, when there's not enough place there either, the menu window goes off screen limits. To see the problem, you can - for instance - run ProcessControler, set the resolution to 800x600, goto Threads and CPU Usage and expand app_server (as shown on the attached screenshot). As far as I know, in this situation the BMenu should move the window adjusting it to the left side of the screen making all its contents visible again. This can be done by a simple fix.

Attachments (2)

menu.png (25.3 KB ) - added by sil2100 13 years ago.
Problem screenshot, ProcessController example
Menu.cpp (42.4 KB ) - added by sil2100 13 years ago.
The small fix

Download all attachments as: .zip

Change History (5)

by sil2100, 13 years ago

Attachment: menu.png added

Problem screenshot, ProcessController example

by sil2100, 13 years ago

Attachment: Menu.cpp added

The small fix

comment:1 by sil2100, 13 years ago

Adding these two lines fixes the problem in the way as stated in the description. Is this how this should work?

comment:2 by axeld, 13 years ago

Resolution: fixed
Status: newclosed

Thanks for the patch, I've applied it in hrev19048. But could you please do a diff next time instead of attaching the whole file? Thanks.

comment:3 by sil2100, 13 years ago

My fault, I'll be more thoughtful next time. Thanks!

Note: See TracTickets for help on using tickets.