Hi, Daniel

Thank you for the patches!

2009/9/24 Daniel Clemente <[email protected]>:
> Hi.
>
> 1. I don't understand why C-t (transpose-chars) won't swap two-letter fields, 
> e.g. og → go. The first patch below makes this work. Note that other programs 
> (readline, bash, Emacs) do the same.

The problem with my initial version (I wrote the original function)
was that it checked for "length <= 2" when it should have checked for
"length < 2". The rationale is that there is no point in trying to
transpose characters on a line with zero or one character. I can't
test your patch, because I don't have Conkeror readily available at
the moment.

> 2. Can transpose-chars be also activated for the minibuffer? The second patch 
> does this. I feel I should have used minibuffer-transpose-chars instead of 
> transpose-chars, just like the other keys do, but I don't know why the 
> minibuffer- prefix was used. In fact, where are all those functions like 
> minibuffer-backward-word defined? Could you add some explanatory comments?

Yes, it should be activated in the minibuffer as well, but I don't
know why the "minibuffer-" prefix on commands exists. A wild guess is
that maybe M-<backspace> should echo "Text is read-only" (like in
Emacs) which doesn't happen in e.g. web text fields.

-- 
Deniz Dogan
_______________________________________________
Conkeror mailing list
[email protected]
https://www.mozdev.org/mailman/listinfo/conkeror

Reply via email to