Add is_visible_in_tree
in TreeItem
#88300
Merged
+44
−11
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Addind a member
is_visible_in_tree
inTreeItem
, which returnstrue
if the item and all its ancestors are visible.This fixes #88251
Each
TreeItem
, after visibility changes, propagate its state to its children.Some uses of
TreeItem::is_visible
are replaced by the newTreeItem::is_visible_in_tree
(I did my best to analyze which ones should be replaced or not).Tested with the minimal repoduction project attached to the corresponding issue #88251
Here is a slightly modified version, where we can turn on and off the visibility of some
TreeItem
with keyboard (1 to 9 normal and numpad keys)Tree Item-is_visible_in_tree.zip
Video captures :
(master)
TreeItem.-.is_visible.bug.mp4
(fix)
TreeItem.-.is_visible.fix.mp4