search

nflath / hungry-delete

Enables hungry deletion in all modes.
98 stars 16 forks source link

hungry-delete is confused by quotes: #16

Closed Wilfred closed 10 years ago

Wilfred commented 10 years ago

Given the Python code, where | is the cursor:

def search_word(request):
    return render({'search_term':search_term,
                   # TODO: i'|
                   'translations': dict(translations_by_lang)})

pressing backspace produces this:

def search_word(request):
    return render({'search_term':search_term,
                   # TODO: itranslations': dict(translations_by_lang)})

I would expect it to just delete the quote.

mwfogleman commented 10 years ago

Yeah, I would expect that, too! I'll try to help look into this soon, as it's quite possible I introduced this bug.

nflath commented 10 years ago

OK, thank you... I thought it wouldn't be expected behaviour, though - isn't backspace usually delete-forward? I'll investigate more when I get home to a keywboard with a backspace button though.

Thanks again.

On Fri, Sep 12, 2014 at 4:52 PM, Michael Fogleman notifications@github.com wrote:

Yeah, I would expect that, too! I'll try to help look into this soon, as it's quite possible I introduced this bug.

— Reply to this email directly or view it on GitHub https://github.com/nflath/hungry-delete/issues/16#issuecomment-55474184.

mwfogleman commented 10 years ago

For some reason, in my Python mode setup, backspace isn't bound to hungry-delete-backward. Can you confirm that that is the case, in your setup, Wilfred?

Wilfred commented 10 years ago

Sorry, I'm blaming entirely the wrong package. I just noticed this bug after installing hungry-delete, but I can reproduce without it. It's a smartparens bug I believe: https://github.com/Fuco1/smartparens/issues/358