Ctrl/Cmd+(Shift)+Z: Undo/Redo

The whiteboard already has Undo/Redo. At least on Mac.
The text widget has some built in support for this
http://tcl.tk/man/tcl8.5/TkCmd/text.htm#M81
Any idea what the keybindings shall be? Mac has Command-Z for undo and
missing key binding for redo.

> Any idea what the keybindings shall be? Mac has Command-Z for undo and
> missing key binding for redo.

Missing? On Tiger I can de Command + Shift + Z for redo...

Added Shift-Z binding for redo in canvas.
Must read up on how this works for the text widget.

Added some test code for the Message text widget. Not exactly sure how it works. It seems to add the separators at newlines.

There already was undo/redo in text but that wasn't activated. A simple -undo 1 was essentially all that was needed. Also sets new separators, fix Mac Edit menu and so on.

This does not work with the latest cvs version on OS X 10.4...

It seems that I was slightly too slow to fix a remaining issue and you was slightly too fast to test it:

revision 1.285
date: 2008/03/25 16:12:35; author: matben; state: Exp; lines: +3 -3
 * Forgot the -undo 1 switch.
----------------------------
revision 1.284
date: 2008/03/25 14:54:25; author: matben; state: Exp; lines: +2 -1
 * Added missing toolbar button for Mac in NewMsg.
 * Bug #204366, Ctrl/Cmd+(Shift)+Z: Undo/Redo
   Added to Edit menu and added undo separators for spaces etc.

It still does not work in the Subject field.

It only applies to text widgets and not to entry widgets like the subject
field. There is no undo stack for entry widgets. If I test Cmd-Z in the
Safari address field it just erases the complete line. Not much of an undo.

1) Subject field != address bar in a browser
2) Even erasing the whole line like how it does in Safari does not work in Coccinella

On Sat, Mar 29, 2008 at 2:21 PM, sander <email address hidden> wrote:

> 1) Subject field != address bar in a browser

It's the same type of widget.

>
> 2) Even erasing the whole line like how it does in Safari does not work in
> Coccinella

I was just questioning the usefulness of it. It could be added to the tk
tracker if you want, but the redo is slightly more complex since I need an
instance variable which isn't so easy in these bindings, but doable, of
course.