Horizontal scrolling is broken in BColumnListView.

[stippi]

Additionally, the scroll area is not updated properly when removing items it seems.

[anevilyak]

Hi stippi,

Want me to take over this one? I have some experience with the BCLV code so I'd be willing. Also, which scroll area are you referring to? Horizontal or Vertical?

[stippi]

Oh yes, please do! Vertical scrolling seems to work fine. But horizontal scrolling does not affect the header view.

[anevilyak]

Will look, do you have any good app for a testcase illustrating this or will I need to write one?

[stippi]

DriveSetup can be used to reproduce it. It also builds in the test environment, in case you want to use that.

[humdinger]

This should be the same bug: ticket #1252

[anevilyak]

Actually, no. BeShare uses the column list view from Santa's Gift Bag, which has nothing in common code-wise with BColumnListView.

[anevilyak]

Hi stippi,

One thing that strikes me as immediately odd is that the ScrollTo hooks don't seem to be getting called....at least the printfs in said hooks aren't showing up at all.

[stippi]

Doh... just change BScrollBar::ValueChanged() to call ScrollTo() instead of ScrollBy(). That should do it. :-)

[stippi]

Thanks for tracking this one down, Rene!

[stippi]

I feel very stupid now, because I was looking at BScrollBar::ValueChanged() _very_ often. :-)

[anevilyak]

If that's the case, shouldn't we modify BView::ScrollBy so it just wraps a call to the virtual version? AFAIK the virtual hook should be called in all cases, otherwise the app can't always get correct behavior, since ScrollBy is not overrideable. Replacing ScrollBy with ScrollTo in ValueChanged does indeed make BCLV behave correctly here btw.

[anevilyak]

Also, this would seem to imply that it is indeed also the cause of ticket #1252, though I'll need to test if that one behaves correctly with that change.

[anevilyak]

Fixed in hrev25559.