Clojurians Log v2

I believe I toggled them off earlier, but I will give it a try again to make sure

please replace asComment with comment https://github.com/BetterThanTomorrow/calva/blob/3f54f2306455399a6f89be5ea560eabc68e72049/src/evaluate.ts#L86

Hi! Just want to give a feedback about new peek-to-result feature. First of all, thanks for putting your effort on this. I see some people likes it, which is great! For me it turned out to be a bit difficult to use: - When evaluating the form with Alt + Enter and having cursor not on a parenthesis, the peek widget displays the current function definition instead of the output result. The result is still accessible, but it takes two extra clicks to reach it. - Closing the peek with Esc does not work if my cursor is inside the code area of the peek (i.e. for scrolling). I think this is a default VSCode behaviour. For now I ended up with just opening the output window with Ctrl + Alt + C O when I want to see full result. It opens in a new editor group and my file is still visible. After some exploration I usually close it again. This workflow is pretty fast (faster than peeking with extra clicks) and can be even faster if I'd be able to close the output window with key shortcut (default VSCode shortcut Ctrl + W is overwritten with Paredit selection in Calva).

New old-repl-window-less vsix https://7173-125431277-gh.circle-artifacts.com/0/tmp/artifacts/calva-2.0.116-711-remove-old-repl-window-5a088397.vsix, it restores evaluate to comment, reported by @elizavetaskull, and also makes the result definition peek be alone when the cursor hasn't moved since the evaluation was run. That means that for a key sequence of alt+enter, alt+f12 , there should only be the results definition there, so no need to navigate. And hover definition on symbols should still be showing what they used to show. Please give it a try, @glebovmaksim and @slack1304 (and everyone, of course).

I am wondering what are the benefits of the output window over the old Repl window? Or is this just for ease of maintenance?

There's a detailed report https://clojureverse.org/t/calva-summer-of-bugs-2020/6303

excellent, a good long read that wont go away, like it does on Slack : )

Totally :spock-hand::skin-tone-2:

How to eval the whole file in editor?

@maxp the command is 'calva - load current file & dependencies', bound by default to ctrl-alt-C Enter

I'm trying to get Calva working with REBL. I haven't read up yet on how the pieces (nrepl/middleware/editor) fit together so am flying a bit blind. I have a deps.edn alias that starts an nrepl, REBL, and middleware (https://github.com/RickMoynihan/nrebl.middleware) to connect these up. Then I connect to that running nrepl from Calva (I'll leave automating with a Calva connection sequence 'till everything else works). This works partially - I can evaluate forms in Calva, and inspect data in REBL. But Calva's documentation and go to definition commands don't work, so presumably the middleware and/or REBL are getting in the way. Is this to be expected, or should I be able to get both data browsing & Calva's commands all working simultaneously?

Pasting seems to be broken for me - the pasted text flashes briefly then disappears.

Generally pasting into an editor? There is a change to the pasting in there, actually. It is a little hard to imagine it having that effect, but somehow it must...

I can confirm that it's broken.

Peeking has become more convenient from a user perspective. Not sure if that little trick with cursor looks good in the code though...

Oh, it fits in there pretty cleanly. We get a position to provide definitions for and just need to check that it's not from where an active results annotation was activated. Like so:

async provideDefinition(document, position: vscode.Position, token) {
    const evalPos = annotations.getEvaluationPosition(position);
    const posIsEvalPos = evalPos && position.isEqual(evalPos);
    if (util.getConnectedState() && !posIsEvalPos) {

Ok, good to know :) Just wondering if we were asking for something that can't be done in a nice way. Thank you!

We're just abusing the definition provider a tiny bit more. Haha.

I've not tried this, but have you read this? https://calva.io/rebl/

Also if goto def is not working, maybe you didn't load cider-nrepl as a dep in your repl

New old-repl-window-less vsix: https://7143-125431277-gh.circle-artifacts.com/0/tmp/artifacts/calva-2.0.116-711-remove-old-repl-window-49cdf856.vsix • Paste now works again • *1. *2, etcetera works as expected in the editors (it never has with Calva, which became exposed with the new output/repl window)

Oh, this should have been in the channel....

New old-repl-window-less vsix: https://7143-125431277-gh.circle-artifacts.com/0/tmp/artifacts/calva-2.0.116-711-remove-old-repl-window-49cdf856.vsix • Paste now works again • *1. *2, etcetera works as expected in the editors (it never has with Calva, which became exposed with the new output/repl window)

With this vsix, I'm still getting the #namespace[namespace] result with *1 :thinking_face:

I eval'ed these two lines one after the other

I might have liked an old vsix. With only the first fix. 😀

I'm afk. But if you trigger a new build, then we should get the *1, etc fix.

Gotcha!

Latest build does not seem to be working either. Maybe you need to re-push?

I built again too, but same commit hash

¯\(ツ)/¯

I did a force push. Might be what is messing it up.

Also having a weird issue where the current namespace keeps reverting back to user in the output window

had to revert back to .115