[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#4981: C-l during query-replace
From: |
Drew Adams |
Subject: |
bug#4981: C-l during query-replace |
Date: |
Mon, 30 Nov 2009 09:24:33 -0800 |
> >> Thanks for fixing this. Are you sure that the new
> >> `recenter-positions' is needed? Given that there
> >> are 3 choices, it's easy to cycle through
> >> them, so adding yet another defcustom that would be use by
> >> a very small number of users does not seem justified (IMHO).
> >
> > I agree that it's overengineering.
>
> I think what is overengineering is adding recenter-top-bottom
> in the first place. It imposes the arbitrary fixed cycling order
> on users with no hope to customize such fundamental feature as
> recentering. `recenter-positions' mitigates this problem in the true
> Emacs way as the *customizable* editor.
As the one originally responsible for `recenter-top-bottom', let me chime in.
;-)
1. Just as with `recenter', a `recenter-top-bottom' user can always provide a
prefix arg to get the exact behavior wanted. It imposes nothing more than
`recenter' imposed.
2. The fact that we seem to be extending the use of this to other areas
indicates that it has proved to be an improvement wrt `recenter'.
3. I have no objection to user's being able, via an option, to add more cycle
points and define their positions. That's not overengineering, IMO. I think the
default should be what `recenter-top-bottom' defined: 3 cycle points, top,
center, bottom.
- Drew
bug#4981: marked as done (C-l during query-replace), Emacs bug Tracking System, 2009/11/30