-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rendering New Notebooks Panel using VS Code Task Provider #1784
Conversation
141ba89
to
a74c7b4
Compare
a74c7b4
to
d63f82d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to not uncollapse the whole tree when switching between named / unnamed @pastuxso?
Screen.Recording.2024-11-20.at.10.39.47.AM.mov
- Introduced caching for notebooks and their cells to optimize tree view rendering. - Added a parent property to RunmeFile to properly associate child items with their parent notebooks. - Updated getChildren method to use cached data and return child items based on their parent. - Removed unnecessary sauceCount logic, simplifying the tree re-rendering workaround. - Added logic to store and preserve the collapsibleState when including named or unnamed cells
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ LGTM. As discussed, any further changes going into new PR(s).
Refactoring of New Notebooks Panel in order to avoid duplication and to have a unique source of truth for tasks.