362738 - restore redo context menu option to textboxes

Status: RESOLVED WONTFIX

(Toolkit :: UI Widgets, enhancement)

Description

[Alex Vincent [:WeirdAl]]

XUL textboxes have an undo context menu option, but not a redo context menu option.  This has bugged me for years, and I don't know why I never bothered to file the bug.

Comment 1

[Alex Vincent [:WeirdAl]]

Uh, oops.  It looks like this was deliberately removed by Blake Ross (http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=textbox.xml&branch=&root=/cvsroot&subdir=mozilla/toolkit/content/widgets&command=DIFF_FRAMESET&rev1=1.9&rev2=1.10).

Can we restore it, please???

Comment 2

[Alex Vincent [:WeirdAl]]

The same was done by Neil in a synchronization move for xpfe:
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=textbox.xml&branch=&root=/cvsroot&subdir=mozilla/xpfe/global/resources/content/bindings&command=DIFF_FRAMESET&rev1=1.31&rev2=1.32

Comment 3

[Alex Vincent [:WeirdAl]]

bug 217420 for anyone interested.

Comment 4

[:Gavin Sharp [email: [e-mail]]]

"Redo" isn't in platform-native textboxes on Windows, so I don't think it should be re-added there. I don't know about other platforms, but unless there's a solid platform-parity case, I think we should stay consistent across platforms and not include it.

Comment 5

[Mike Connor [:mconnor]]

Gavin's right, that's why we removed it.  In the absence of a reason for adding it, marking WONTFIX.