On Aug 18, 9:19 am, "Edward K. Ream" <[email protected]> wrote:
> On Tue, Aug 17, 2010 at 8:22 PM, Sam Hartsfield <[email protected]> wrote:
> - Delete just the selection.
> - Delete all words covered by any part of the selection.
> However, edge cases here are tricky.
> - Delete the word at the forward edge of the selection for delete-word,
> and back edge of the selection for backward-delete-word.
> - Delete the word at the *back* edge of the selection for delete-word,
> and the *forward* edge of the selection for backward-deleted-word.
One more: a behavior for many editors is to delete from the *cursor*,
whether that's at the beginning or end of the selection. Depending on
the editor, the selection may or may not be maintained.
But I agree that the choice is unimportant, and I'm happy with your
current implementation.
> The new code and unit tests are on the trunk at rev 3266.
A couple of minor points that I don't think warrant me creating a
branch:
u = c.undoer
is unused, and
c.editCommands.moveWordHelper(None,extend=False,forward=forward)
I think should be
self.moveWordHelper(None,extend=False,forward=forward)
from looking at similar methods.
--
You received this message because you are subscribed to the Google Groups
"leo-editor" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/leo-editor?hl=en.
