61341 - Alt+Bksp shortcut for Undo

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P4)

Description

[[e-mail]]

Alt+BackSpace is one of the "old" Windows shortcuts, along with Ctrl+Delete for
copy, Shift+Insert for paste and Shift+Delete for cut ( except that it deletes
instead, but that's another bug )-:

The code in platform*Bindings.xul would look something like this:

<key id="winAlternateUndoKey" keycode="VK_BACK"
 oncommand="
  var controller = document.commandDispatcher.getControllerForCommand('cmd_undo');
  controller.doCommand('cmd_undo');" modifiers="alt"/>

Comment 1

[rubydoo123]

setting to future, assigning to brade for review -- kathy reassign as appropriate

Comment 2

[Kathleen :Brade]

accepting bug; I'll get to this when I get back from maternity leave probably

cc akkana; akkana--do we want this keybinding on unix as well (I think we have 
the other ones there)

Comment 3

[Akkana Peck]

I'm not clear why we define any of those "old windows shortcuts" on Unix in the
first place.  The only person who can answer this is whoever decided that we
should have the others on Unix.  Ben?  Was that you, or do you know who made the
decision?

Comment 4

[Jesse Ruderman]

This is a standard shortcut on Windows.  Clearing [rfe] from summary and 
raising severity from trivial to minor.

Comment 5

[timeless]

wow, it may be a standard, but, to paraphrase alecf, i never new that. [or at 
least i can't remember using it since 1995]

Comment 6

[Kathleen :Brade]

Attachment: add keybindings for undo to input, textarea, editors

Comment 7

[Kathleen :Brade]

Note:  I haven't had a chance to actually try this patch in a Windows build
(since I don't have one today).

Comment 8

[Kathleen :Brade]

cc kin for sr=

Comment 9

[kinmoz]

The 02/28/01 13:33 fix seems to work in TextWidgets, but not Composer or MsgCompose.

Comment 10

[Kathleen :Brade]

This bug is fixed but won't appear to work in Composer until bug #71326 is fixed.

Comment 11

[sujay]

will verify after 71326 is fixed.

Comment 12

[sujay]

verified in 3/22 build.