Closed
Bug 129916
Opened 22 years ago
Closed 22 years ago
Add Accel+shift+z for redo on all platforms
Categories
(Core :: DOM: UI Events & Focus Handling, defect, P1)
Core
DOM: UI Events & Focus Handling
Tracking
()
VERIFIED
FIXED
mozilla1.0
People
(Reporter: aaronlev, Assigned: aaronlev)
Details
Attachments
(2 files, 1 obsolete file)
4.33 KB,
patch
|
mozilla
:
review+
|
Details | Diff | Splinter Review |
4.28 KB,
patch
|
Brade
:
review+
alecf
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
Netscape is adding this keybinding because of a confidential customer requirement; if Mozilla would like to be compatible, here's the change, otherwise we'll have to take it in our tree only. Our normal binding for redo, Accel+Y will remain for now. Accel+shift+Z is actually used for redo in a variety of Windows programs, including notepad, Visual Studio, and ordinary text fields. There is no clear standard for redo, except that it can be Accel+Y or Accel+shift+Z.
Assignee | ||
Updated•22 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla1.0
Assignee | ||
Comment 1•22 years ago
|
||
Updated•22 years ago
|
OS: Windows 2000 → All
Hardware: PC → All
Comment 2•22 years ago
|
||
Why doesn't this patch just remove it from the Mac file and put it in the cross-platform file?
Assignee | ||
Comment 3•22 years ago
|
||
You're right, it should be in all of them. Call me crazy, I don't remember what my original reason was.
Assignee | ||
Comment 4•22 years ago
|
||
Attachment #73433 -
Attachment is obsolete: true
Comment 5•22 years ago
|
||
Comment on attachment 73580 [details] [diff] [review] Puts everything in htmlBindings.xml instead of platform specific files r=jgaunt
Attachment #73580 -
Flags: review+
Comment 6•22 years ago
|
||
in the Mac-specific file, you added some spaces (tabs?) after <handlers> please remove those so diff wouldn't show that you touched that line.
Assignee | ||
Comment 7•22 years ago
|
||
Comment 8•22 years ago
|
||
Comment on attachment 73583 [details] [diff] [review] Whitespace to brade's specification r=brade (although you *can* remove the blank line if you want)
Attachment #73583 -
Flags: review+
Updated•22 years ago
|
Attachment #73583 -
Flags: superreview+
Comment 9•22 years ago
|
||
Comment on attachment 73583 [details] [diff] [review] Whitespace to brade's specification sr=alecf
Comment 10•22 years ago
|
||
Comment on attachment 73583 [details] [diff] [review] Whitespace to brade's specification a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #73583 -
Flags: approval+
Assignee | ||
Comment 11•22 years ago
|
||
checked in
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 12•22 years ago
|
||
vrfy'd fixed with 2002.07.23-1.0 comm branch bits. linux rh7.2, win2k: both shift+ctrl+Z and ctrl+Y behave as redo. mac os x 10.1.5: only shift+cmd+Z behaves as redo. (if this isn't expected, feel free to reopen.)
Status: RESOLVED → VERIFIED
Updated•5 years ago
|
Component: Keyboard: Navigation → User events and focus handling
You need to log in
before you can comment on or make changes to this bug.
Description
•