Michael Overmeyer

Results 73 comments of Michael Overmeyer

@riccardoferretti Here's a reformulation. Let me know what you think. This is the same logic as described in the original proposal, but fills in the edge cases that were glossed...

> Is this semantically what you meant? @riccardoferretti We're in agreement with the first bunch of lines: ```typescript function newNoteInformation(fallbackContent: string, fallbackFilename: string) { let content: string, filepath: Uri; //...

> Point B: I have consolidated `loadFromTemplate` and `loadFilepathFromTemplate`. Was there a reason to separate the two that I am missing? Yeah, looking back, I think that was just a...

> Thoughts? @riccardoferretti I think I've convinced myself that splitting `foam.rootDirectory` into two concepts makes sense. I've written a stream of consciousness below that walks through my thinking. I'd appreciate...

@riccardoferretti I think its accurate to say you've been trying to model `` (and `foam.notesDirectory`) as a variable that gets resolved based on the current environment. I'm probably missing something,...

@riccardoferretti and I chatted in real-time and realized that I was getting hung up on trying to handle all the scenarios automatically, which was causing complexity. The breakthrough was the...

@riccardoferretti > I assume there is a type and you meant: There is a workspace open, but no files are open right? Yes. Fixed. ---- > If the two of...

@riccardoferretti > What are your thoughts? I like it. I feel that our ideas are getting very close to each other. Some comments (all discussion, no major criticisms) ### Questioning...

I like the idea. --- @riccardoferretti If you use `${FOAM_SELECTED_TEXT}`, you end up with a collision if the user had already selecting some text. So I'm not sure that we'd...

@riccardoferretti That's very reasonable. I agree that ideally we wouldn't bother with creating a new variable for this, and I agree that the use case for using both is a...