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.
🦟 Bug fix
Summary
Problem: API to change visibility of the ArrowVisual's Head visual was affecting the incorrect child visual's visibility - it updated the Rotation visibility instead. This PR fixes indexing of child nodes in ArrowVisual.
More info:
The index of child nodes have changed after #831 that changed internal data structure from map to vector. With that change, the index number now corresponds to the order in which the child nodes are added.
Updated test to make sure when we request for ArrowVisual's Head, Shaft, and Rotation child visuals, it returns the visual with the correct geometry.
Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.