From 7e80111d926ec7b2f6f912c52af2d02171ffd761 Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Tue, 9 Nov 2021 08:49:21 -0700 Subject: [PATCH] Canvas: Fix notification to parent of show/hide status This was changed in fc9840f2a96a to hard code a "false" argument describing whether or not the child bbox had changed. This causes the parent to not recompute its own bounding box when a child is shown/hidden, which is incorrect. We now force it to true, which may be overkill for some purposes, but at worst will just force an unnecessary bbox re-computation and nothing more. --- libs/canvas/item.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libs/canvas/item.cc b/libs/canvas/item.cc index 572fe78e90..ccef3a0408 100644 --- a/libs/canvas/item.cc +++ b/libs/canvas/item.cc @@ -404,7 +404,7 @@ Item::propagate_show_hide () /* bounding box may have changed while we were hidden */ if (_parent) { - _parent->child_changed (false); + _parent->child_changed (true); } _canvas->item_shown_or_hidden (this);