[haiku-development] Re: BControlLook scrollbar APIs

  • From: waddlesplash <waddlesplash@xxxxxxxxx>
  • To: Haiku Development ML <haiku-development@xxxxxxxxxxxxx>
  • Date: Sun, 22 Jul 2018 09:26:39 -0400

On Sun, Jul 22, 2018 at 3:15 AM, Adrien Destugues <pulkomandy@xxxxxxxxxxxxx>
wrote:

BScrollBar has its own _ButtonFor code, but in the current state this
would not work with a replacement BControlLook with different button
sizes (or no buttons at all, or…). The code in _ButtonFor would get out
of sync with the one in BControlLook.


We have two options here:
- Either revert to separate DrawButtons and DrawBackground methods, and
  let BScrollBar manage the layout. In this case, BControlLook cannot
  change the button layout of the scrollbar

This doesn't sound like a good solution. What if someone writes a new
BControlLook with different scrollbar sizing?

- Or, let BControlLook tell BScrollBar where the buttons should be
  placed.

I think this is the better solution, yes.
Sorry for the late comments, I had no time to review the BControlLook
version of https://review.haiku-os.org/#/c/haiku/+/300/ which spent just
6 hours on review before being +2 and merged by the same person (I would
prefer we avoid that, unless there is an emergency).

Huh? The timestamps on Gerrit show that John posted the version with
BControlLook changes at around 19:00, and I merged it around 12:00 the next
day. So that's a difference of 17 hours or so.

Further, John has commit access, which means that I trust him more, and
that if there is a problem with the patch, it'll be his problem, not mine.
:-P

-waddlesplash

Other related posts:

  1. Home
  2. haiku-development
  3. Archive
  4. 07-2018