113551 - In Mail Composer, Copy&Paste keys on Sun keyboard do not work

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect)

Description

[Jay Yan]

From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 5.01; Windows NT 5.0)
BuildID:    

When composing Email, two keys: Copy and Paste on Sun keyboard do not work.

Reproducible: Always
Steps to Reproduce:
1.Open Mozilla
2.Tasks->mail&Newsgroups
3.Compose a new mail
4.press Copy and Paste key to test the function

Actual Results:  Nothing happens

Expected Results:  If Copy key is pressed, the selected content should be 
copied into clipboard, and then if Paste key is pressed, the content should 
copied from clipboard to the current position.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Jay, are the keys supposed to use Primary or Clipboard?

Comment 2

[Akkana Peck]

Our key event model doesn't support keys which exist only on some platforms, so
there's no way to express these keys in a mozilla key binding.  
Bug 36640 tracks this issue (it also applies to keys such as the Windows key)
and there's a longdiscussion there about the sun bindings (though the last
comment says they work ...?)

*** This bug has been marked as a duplicate of 36640 ***

Comment 3

[sujay]

verified.

Comment 4

[Jay Yan]

Attachment: The diff file making SUN Copy/Paste/Cut Keys work well

This bug is different with 36640:
1  36640 only fixed problem of SUN specific Copy&Paste&Cut keys partially,
users can not use these three keys while editing the mail body of a new
mail(these keys work well on the CC list field and Subject field)

2 The fix of 36640 is out-of-dated, because the xbl files are upgraded now.

This patch can make SUN Copy/paste/Cut work well.I have tested it on Tier-1
platforms. It can be reviewed now.

Comment 5

[Jay Yan]

Reopen this bug( please see the attach file) 
and add Margaret to the CC list.

Comment 6

[kinmoz]

Reassigning to akkana ... akk if the keybindings in the patch look ok to you, 
consider it sr=kin.

Comment 7

[Akkana Peck]

Okay, I don't have a problem with this, and it shouldn't conflict with any other
keys.  Cc'ing other key binding people who need to know about this, in case
anyone has an objection; if I don't hear any nays, I'll check it in.  (I can't
test it -- my keyboard doesn't have those keys, so I'll take your word for it
that it works.)

Comment 8

[Boris Zbarsky [:bzbarsky]]

I just tested this on Mozilla/Linux displayed remotely to a Solaris machine
(which has the cut/copy/paste keys).  Work like a charm with this patch.

Comment 9

[Margaret Chan]

Looks like there is no objection, [e-mail]:  Would you mind checking
this in please?  Thanks.

Comment 10

[Akkana Peck]

It's in.

Comment 11

[Margaret Chan]

Thank you.  Margaret

Comment 12

[sujay]

Jay/Margaret, can you please verify this and mark verified-fixed ? thanks.

Comment 13

[Margaret Chan]

Jay:  Can you verify this bug and mark it as verified please?  Margaret

Comment 14

[Jay Yan]

verified, 
Thanks, Margaret Akkana and Sujay