marimo icon indicating copy to clipboard operation
marimo copied to clipboard

feat: added expandAll function to CollapsibleTree to handle expansion of all nested nodes

Open bjoaquinc opened this issue 7 months ago • 1 comments

📝 Summary

Created a expandAll function to handle expansion of all nodes including nested children inside a column. Partially resolves #4163 by adding a way to expand all nested collapsed nodes

🔍 Description of Changes

  • Created expandAll() function in id-tree.tsx
  • Created a test for expandAll() in id-tree.test.ts to ensure that it expands nested collapsed nodes properly

📋 Checklist

  • [x] I have read the contributor guidelines.
  • [x] For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • [x] I have added tests for the changes made.
  • [x] I have run the code and verified that it works as expected.

📜 Reviewers

@mscolnick

bjoaquinc avatar May 01 '25 00:05 bjoaquinc

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

Name Status Preview Comments Updated (UTC)
marimo-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 1, 2025 3:39pm

vercel[bot] avatar May 01 '25 00:05 vercel[bot]

i tried to make a simplification but realized it was wrong, so I reverted my change. this looks good!

mscolnick avatar May 01 '25 15:05 mscolnick

🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.13.4-dev11

github-actions[bot] avatar May 01 '25 16:05 github-actions[bot]