[haiku-development] Re: tracker patch
- From: "Michael Berg" <mike@xxxxxxxxxxx>
- To: haiku-development@xxxxxxxxxxxxx
- Date: Tue, 29 May 2007 14:38:00 -0400
I've looked at the code a bit, here's what I see.
The team items are held in TExpandoBar. Which is descended from BMenuBar.
BMenuBar is a customized BMenu. BMenu's as top level or 'root' menus do
not scroll.
So in order to get TExpandoBar to scroll we have to either change BMenuBar
to allow for scrolling or introduce a new view that TExpandoBar would sit in
and be scrolled by. I think that the BMenu, BMenuBar and BMenuItem
hierarchy would benefit from the changes but I don't know what others would
think to reworking the internals of them. I think that adding scrolling to
TExpandoBar directly would become a very ugly hack. Adding a new view into
the mix, while very basic could hinder performance.
The best bet would be to reimplement BMenu as a BMenuItem descendant. This
would give BMenu scrolling inherently. It would also increase the
flexibility of BMenus as a whole. They could be hosted in other controls,
like toolbars. With replicants they could be integrated into the host
applications own menu for full control of the replicant.
Some additional properties would need to be added to BMenuItem to indicate
it as a top level menu for iteration and insertion purposes. But on the
whole I feel it would be beneficial and worth the effort.
- Follow-Ups:
- [haiku-development] Re: tracker patch
- From: Ingo Weinhold
- References:
- [haiku-development] Re: tracker patch
- From: Michael Berg
- [haiku-development] Re: tracker patch
- From: Axel Dörfler
- [haiku-development] Re: tracker patch
- From: Michael Berg
Other related posts:
- » [haiku-development] tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- » [haiku-development] Re: tracker patch
- [haiku-development] Re: tracker patch
- From: Ingo Weinhold
- [haiku-development] Re: tracker patch
- From: Michael Berg
- [haiku-development] Re: tracker patch
- From: Axel Dörfler
- [haiku-development] Re: tracker patch
- From: Michael Berg