[interfacekit] Re: PATCH: BView::IsHidden()

> Hi,
>
> I've added the missing BView::IsHidden(BView *lookingFrom) method, and
> fixed the former BView::IsHidden() method.
> I also couldn't resist and applied our style guide to those two methods
> to let the idea of its existence grow in your mind (slowly but
> steadily).
OK

> And, I commented one "ToDo:" item in BView::Hide(), too.

// TODO: You may hide it by relocating with -17000 coord units to the
left???
// "I don't think you want to do that" - axeld.

    Be did that!

Please let the TODOs at the start of the line! That way they "glow"
remembering me there is something to be done.

> If you'd be okay, I'd happily commit the patch.
I'm OK.


Adi.


Other related posts: