Open JohnGemstone opened 1 month ago
Overall, everything looks good. I made sure to test it out. There are a few small things like padding/scrolling that need to be fixed when the view is embedded, but this is a great MVP for this feature.
Thanks for upgrading the Typescript version to 5.6.x. That actually solves an error that I was getting on build because of the verbatimModuleSyntax
flag.
I added a few comments on things that I think could be improvement. Thanks for your contribution and diving into Svelte!
@JohnGemstone I am sorry, I thought I had finished the review.
This feature allows users to access Vault-Explorer views directly within their notes, streamlining the process of navigating between views without needing to go to the side panel. Users can define group views as usual and then reference them for embedding like so: