[haiku-commits] Re: haiku: hrev50780 - src/kits/interface

  • From: Stephan Aßmus <superstippi@xxxxxx>
  • To: <haiku-commits@xxxxxxxxxxxxx>
  • Date: Wed, 21 Dec 2016 08:32:22 +0100

Hi,


Am 21. Dezember 2016 4:38:00 vorm. schrieb anevilyak@xxxxxxxxx:

hrev50780 adds 1 changeset to branch 'master'
old head: a835815beb2ec4ed9ae9c41ea7408279fbdd44fa
new head: 078b08e4edc6911cd7515980d372686590815bbd
overview: http://cgit.haiku-os.org/haiku/log/?qt=range&q=078b08e4edc6+%5Ea835815beb2e

----------------------------------------------------------------------------

078b08e4edc6: BListView: Fix missing update for list items.

  - According to the Be Book, one of BListItem::Update()'s responsibilities
    is updating the item's internal height/width. However, while we were
    providing such updates in the case of the item initially being added
    to the view, or when the list view's font was changed, we were not doing
    so on frame resize. This would lead to items having a stale width field
    in such a circumstance, leading to possibly incorrect behavior in apps
    relying on that field to be correct.

----------------------------------------------------------------------------

diff --git a/src/kits/interface/ListView.cpp b/src/kits/interface/ListView.cpp
index cd12e3f..59f9d40 100644
--- a/src/kits/interface/ListView.cpp
+++ b/src/kits/interface/ListView.cpp
@@ -257,6 +257,9 @@ void
 BListView::FrameResized(float newWidth, float newHeight)
 {
        _FixupScrollBar();
+
+       // notify items of new width.
+       _FontChanged();
 }

Can the private method be renamed? Such as _UpdateItems()?

Best regatds,
Stephan




Other related posts: