REPL completions for brackets and quotes.

[KristofferC]

Trying to upstream a bit more from https://github.com/KristofferC/PimpMyREPL.jl

This adds some convenience bracket completions that are common in editors. I basically replicated the behaviour I get from my editor that I like.

Example:

[yuyichao]

FWIW, this is one of the Jupyter notebook features I hate the most...... (p.s. the other one being the multiple input cursor...)

[KristofferC]

Hah! Hmm, maybe make it optional then... I like it because it makes is easy to balance brackets.

[yuyichao]

And a little more detail about why I hate this feature, (just tried it again to remind myself about it).

When the input is (()<cursor>) (with possibly more stuff between the opening parenthesis), typing <backspace>) will end up net deleting a ) and move the cursor after the (originallly) second closing parenthesis. I've got countless number of parenthesis mismatch after editing a cell because of this.

[KristofferC]

So it is just the "only move to the right if matching right bracket"-part you dont like? However, it would be annoying if f, (, x, ) turned into f(x)) so they sort of tie together.

[yuyichao]

only move to the right if matching right bracket

Yes, that's the main reason. I can easily imagine that a different heuristic can be annoy in other cases though.

Since this is strictly a negative feature for me (I always type matching parentheses/quotes), it'll be pretty hard to implement this that doesn't get in my way from time to time. On the other hand, I agree this is a nice feature and I'll be happy as long as there's a way to turn it off (I'm ok with this being the default if more people like it that way).

[KristofferC]

Ok, I will make sure it is easy to enable/disable and then we can think about what should be default.

[KristofferC]

Ok, pushed new version with a function to disable it + NEWS entry.

[KristofferC]

Also this is only for the Julia REPL now. I could make it the default fall back for all REPLs, not sure what is best.

[KristofferC]

The emoji competition suggests that maybe this should be off by default :P

[omus]

Should matching also apply to back ticks?

[KristofferC]

This seems to generally not be very desired and is already implemented in a package so probably not needed in the default Base REPL.