Undo/Redo menu items are not properly enabled/disabled in Mail app

[ksmith]

When you launch the mail app, the redo menu item in the Edit menu is enabled. This should not be enabled. There is nothing to redo. The undo menu item is properly disabled at this point. Type a character in the message body. Undo is now enabled. Select undo, which will remove the character you typed. The undo option remains enabled, even though there is nothing else that can be undone. The undo option should become disabled when there is nothing remaining to be undone.

[ksmith]

After further investigation, this seems to be the result of how undo works in the BTextView, which uses a BUndoBuffer. The BUndoBuffer never resets its state to B_UNDO_UNAVAILABLE when the undo buffer is empty, which is why the Undo menu item becomes permanently enabled.

The Redo menu item is just enabled. There is no check for the availability of a Redo.

[jackburton]

Replying to ksmith:

After further investigation, this seems to be the result of how undo works in the BTextView, which uses a BUndoBuffer. The BUndoBuffer never resets its state to B_UNDO_UNAVAILABLE when the undo buffer is empty, which is why the Undo menu item becomes permanently enabled.

The Redo menu item is just enabled. There is no check for the availability of a Redo.

Does this applies to R5 too ?

[ksmith]

Replying to jackburton:

Does this applies to R5 too ?

Yes. This is the way undo works under R5 (at least in the BeMail app).