From a7c240ff0265446ee30b8e5f28f2ef4bd8e60912 Mon Sep 17 00:00:00 2001 From: PanteraPolnocy Date: Fri, 27 Mar 2026 19:41:36 +0100 Subject: [PATCH] FIRE-36479 Probable fix for nested scrollbars in outfits panel LLFlatListView::sort() rearranged items via rearrangeItems(), but never notified the parent accordion tab about the updated content height. When outfit items were loaded without an active filter this notification was skipped entirely, so that left the flat list stuck with an outdated size. Because LLFlatListView inherits from LLScrollContainer, it ended up showing its own scrollbar inside the accordion's scrollbar - hopefully fix it by calling notifyParentItemsRectChanged() after rearrangeItems() in sort(). The call is guarded by mPrevNotifyParentRect, so redundant calls should be safe. Now... would it scale with thousands of outfits that some residents have? Let's see in the nightlies... Signed-off-by: PanteraPolnocy --- indra/llui/llflatlistview.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/indra/llui/llflatlistview.cpp b/indra/llui/llflatlistview.cpp index 32f456f56a..0716925c6a 100644 --- a/indra/llui/llflatlistview.cpp +++ b/indra/llui/llflatlistview.cpp @@ -443,6 +443,7 @@ void LLFlatListView::sort() mItemPairs.sort(ComparatorAdaptor(*mItemComparator)); rearrangeItems(); + notifyParentItemsRectChanged(); // FIRE-36479 Multiple nested scrollbars in Appearance window } bool LLFlatListView::updateValue(const LLSD& old_value, const LLSD& new_value)