[haiku-commits] Re: r42120 - haiku/trunk/src/kits/interface

  • From: Stephan Aßmus <superstippi@xxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sun, 12 Jun 2011 07:39:23 +0200

Am 12.06.2011 06:11, schrieb Ryan Leavengood:
On Sat, Jun 11, 2011 at 10:27 PM,<anevilyak@xxxxxxxxx>  wrote:
Remove unnecessary calculations from BListView::ScrollToSelection(). At least I
see no useful purpose to the distinction made here.

Yeah it seems silly to scroll the item to the top if the item is above
the view-port and to the bottom if it is below. Maybe this was just
pure BeOS behavior copying here?

Why do you guys consider it useless? The point of this implementation was to scroll the smallest possible amount. Potentially it means that some items that were visible before are still visible after the scroll, so that the user has a reference. What was the problem that was fixed by removing this feature?

Best regards,

Other related posts: