Opened 12 years ago

Closed 12 years ago

Last modified 12 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 12 years ago.
Problem screenshot, ProcessController example
Menu.cpp (42.4 KB) - added by sil2100 12 years ago.
The small fix

Download all attachments as: .zip

Change History (5)

Changed 12 years ago by sil2100

Attachment: menu.png added

Problem screenshot, ProcessController example

Changed 12 years ago by sil2100

Attachment: Menu.cpp added

The small fix

comment:1 Changed 12 years ago by sil2100

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

comment:2 Changed 12 years ago by axeld

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 Changed 12 years ago by sil2100

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

Note: See TracTickets for help on using tickets.