[Bf-blender-cvs] [084f072aae7] master: Fix T73191: Buttons in lower left of Preferences broken

Julian Eisel noreply at git.blender.org
Wed Jan 22 18:03:07 CET 2020


Commit: 084f072aae73de39fecc35ac97d602b1db7d0d98
Author: Julian Eisel
Date:   Wed Jan 22 17:54:18 2020 +0100
Branches: master
https://developer.blender.org/rB084f072aae73de39fecc35ac97d602b1db7d0d98

Fix T73191: Buttons in lower left of Preferences broken

I'm still not entirely sure what was going on - I know that the execute
region didn't get initialized correctly, but doing that at a later point
didn't fix the issue. Apparently forcing the header region to
re-initialize does fix it, even though I was sure this was redundant.

Also fixes a memory leak in UI code after preferences were opened.

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

M	source/blender/editors/screen/screen_ops.c

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

diff --git a/source/blender/editors/screen/screen_ops.c b/source/blender/editors/screen/screen_ops.c
index 9ea8c8293d4..0e1ae03fd20 100644
--- a/source/blender/editors/screen/screen_ops.c
+++ b/source/blender/editors/screen/screen_ops.c
@@ -4843,7 +4843,10 @@ static int userpref_show_invoke(bContext *C, wmOperator *op, const wmEvent *even
      * So hiding in the temp window makes sense. */
     ScrArea *area = CTX_wm_area(C);
     ARegion *region = BKE_area_find_region_type(area, RGN_TYPE_HEADER);
+
     region->flag |= RGN_FLAG_HIDDEN;
+    ED_region_visibility_change_update(C, area, region);
+
     return OPERATOR_FINISHED;
   }
   else {



More information about the Bf-blender-cvs mailing list