p3r7 / awesome-elisp

🏵️ A curated list of Emacs Lisp development resources
The Unlicense
761 stars 33 forks source link

Transform code navigation to more general code editing section #13

Closed clemera closed 3 years ago

p3r7 commented 3 years ago

Yeah, good point, this section started to feel busy.

Some stuff I'm less fond of:

clemera commented 3 years ago

elisp-docstring-mode: numerous broader package do this (smart-parens, lispy...)

I did not know that, they also let pop you to a separate editing buffer with a major mode for editing docstrings?

speed-of-thought-lisp: mhm, sound just like an elisp-exclusive yasnippet, right? Don't know if I'd recommend that over the more generic solution.

It is different, I find it invaluable to edit Elisp. The expansion is automatic (no extra key you just type Space like with abbrevs) and usually it doesn't interfere with for example variable names because it only expands function names in appropriate places.

even though it's not Elisp-sepcific but more Lisp-sepcific I also thought of including lispy. But not without paredit and paxedit. All three would deserve their own sub-section (Editing ?) as they are mostly about the whole editing experience than refactoring per se.

Okay, I'm unsure should I remove lispy for now or add the other two?

p3r7 commented 3 years ago

elisp-docstring-mode: [...] they also let pop you to a separate editing buffer with a major mode for editing docstrings?

Oh ok. I only read the description you wrote (Handles escaping and unexcaping of quotes automatically). After going on the package repo I see more clearly what it does. We'll need to update the description to make that more apparent.

speed-of-thought-lisp: [...] it doesn't interfere with for example variable names because it only expands function names in appropriate places

Oh yeah, I see. yasnippet is dumber than that. Theoretically it could be context-aware with conditions but in practice default snippets (from yasnippet-snippets) don't implement those.

I'll need to try it out, then ;)

lispy: [...] Okay, I'm unsure should I remove lispy for now or add the other two?

I'll merge it like this for now and add the 2 others in a second step.

clemera commented 3 years ago

Thanks for the review!

p3r7 commented 3 years ago

You're welcome :smile:

Don't hesitate to challenge my submissions as well, if you ever find something fishy.

From your last edits, I've created a dedicated section for lispy / paxedit / paredit.

I've also extended the description of speed-of-thought-lisp and of elisp-docstring.

clemera commented 3 years ago

Looks good, very nice! I think calling yasnippet an equivalent of speed-of-thought could give a wrong impression. Yasnippet is much more powerful and different in scope so maybe we could think about rephrasing that a bit.

p3r7 commented 3 years ago

maybe we could think about rephrasing that a bit

Did a bit of rephrasing. Tell me how that sounds now.

clemera commented 3 years ago

Sounds great!