Fixed displayed values ordering

[harnish7576]

Description

Fixes #2342

• Sorting naively created sorted list which had entirely new objects.

• SortTreeInPlace happens before rendering, modifying the original objects.
• This way it only affects the order, not object structure.
• StackedMenu now receives the same object references which preserves interactivity.

Changes

• Added a SortTreeInPlace Function that sorts the same objects in memory instead of creating new ones and Ensures children are sorted recursively.
• Sorted based on Labels instead of Name

Screenshots

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
policyengine-app	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 27, 2025 4:11am

[anth-volk]

Thanks for your work on this @harnish7576! A couple things:

• Please add "Fixes #NUMBER" as the first item in the PR description
• Please rename the issue as something readable (e.g., "Fixed displayed values ordering")

[harnish7576]

@anth-volk I've updated the PR as requested.

[anth-volk]

Hey @harnish7576, thanks for this! I've requested one minor change, then should be good to merge.

[anth-volk]

Thank you for this @harnish7576! Glad to get this merged.