[Bf-blender-cvs] [5650468c842] blender-v2.91-release: Property Search: Fix missing update switching tabs after search

[Hans Goudey]

Commit: 5650468c842eebef01c4959965b48dde686b8328
Author: Hans Goudey
Date:   Tue Nov 3 16:24:22 2020 -0600
Branches: blender-v2.91-release
https://developer.blender.org/rB5650468c842eebef01c4959965b48dde686b8328

Property Search: Fix missing update switching tabs after search

What I thought was an "optimization" was really a bug. The "use search
for expansion" value needs to be set for every panel, even panels in
other tabs. Otherwise it won't be properly set when switching back to
a tab that was visited during search.

Differential Revision: https://developer.blender.org/D9427

===================================================================

M	source/blender/editors/interface/interface_panel.c

===================================================================

diff --git a/source/blender/editors/interface/interface_panel.c b/source/blender/editors/interface/interface_panel.c
index 2a95a34e5e7..bfe5dc223c8 100644
--- a/source/blender/editors/interface/interface_panel.c
+++ b/source/blender/editors/interface/interface_panel.c
@@ -938,11 +938,7 @@ static void region_panels_set_expansion_from_seach_filter(const bContext *C,
                                                           const bool use_animation)
 {
   LISTBASE_FOREACH (Panel *, panel, &region->panels) {
-    /* Checking if the panel is active is only an optimization, it would be fine to run this on
-     * inactive panels. */
-    if (panel->runtime_flag & PANEL_ACTIVE) {
-      panel_set_expansion_from_seach_filter_recursive(C, panel, use_search_closed, use_animation);
-    }
+    panel_set_expansion_from_seach_filter_recursive(C, panel, use_search_closed, use_animation);
   }
   set_panels_list_data_expand_flag(C, region);
 }