114065 - Smoketest doc issues: Intl Composer: cut &paste from context menu

Status: RESOLVED FIXED

(SeaMonkey :: Composer, defect)

Description

[Jay Farrell]

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

Test IC.2

  This test requires that you type in some Japanese characters. 
  I wasn't sure how to do this, but what I did was open 
  www.zdnet.co.jp and then right click | edit Page. This allowed 
  me to copy some of the japanese characters so I could past them 
  in my original Composer window. 
 
  The problem is that I could only paste these characters in the 
  window by doing clicking on Edit | Paste; it did NOT work when 
  I right clicked on my mouse and selected paste.  The browser 
  window worked fine both ways. 


Reproducible: Always
Steps to Reproduce:
1. Follow steps for IC.2
2. open a window to www.zdnet.co.jp and right click | edit page
3. copy some japanese characters to paste in you Compose window
4. try to paste into composer window with mouse right click | paste

Actual Results:  Does not work with mouse paste

Expected Results:  Mouse paste should work

This works the same on OpenVMS as it does on Linux. Not sure if this
should be documented in the test, or fixed. I figured Asa might be
able to determine this better than I.

Comment 1

[Asa Dotzler [:asa]]

sounds like a bug to me. over to Composer. 

Comment 2

[Kathleen :Brade]

-->brade
Is this still a bug on recent builds?

Comment 3

[Kathleen :Brade]

Is this still a bug?

Comment 4

[Kathleen :Brade]

is this still a bug?

Comment 5

[Teruko Kobayashi]

Kasumi, would you try to reproduce this?
Thanks,
Teruko

Comment 6

[Kasumi]

Changed QA contact to myself.  

Comment 7

[Kasumi]

I can't reproduce on 2002-06-05-08-1.0.0 on Win XP Pro. JA.

Comment 8

[Arunan Balasubramaniam]

I am seeing this on 2002090804 W2K, with English characters.

Select some text, copy to clipboard.  Confirmed it pastes into notepad.  Does
not paste into a new composer window from the context menu.  Does paste from the
main menu.

Comment 9

[Charles Manske]

I'm looking at this now

Comment 10

[Charles Manske]

Attachment: patch v1

restore doCommand calls in Composer context menus

Comment 11

[Charles Manske]

This is a regression caused by fix to bug 164647

Comment 12

[Kathleen :Brade]

Comment on attachment 98593 [details] [diff] [review]
patch v1

r=brade but add comment explaining why clipboard items require calling
"goDoCommand" in addition to observes attribute and remove extra space after: 
observes="cmd_redo"

Comment 13

[Charles Manske]

Ok. Requests will be done.

Comment 14

[Joe Hewitt (gone)]

Comment on attachment 98593 [details] [diff] [review]
patch v1

sr=hewitt

Comment 15

[Charles Manske]

checked into 1.2beta trunk

Comment 16

[[e-mail]]

Comment on attachment 98593 [details] [diff] [review]
patch v1

Is there any benefit in using command="cmd_undo" instead of
oncommand="goDoCommand('cmd_undo')" other than the perf/bloat saving by not
constructing extra event handlers?

Comment 17

[Charles Manske]

I tried using "command=" when I fixed bug 164647, but then the menuitems didn't
enable/disable properly. It seemed I had to use "observes=" for that to work,
so I thought using "doCommand()" was the appropriate thing to use in that 
context.

Comment 18

[Charles Manske]

*** Bug 166738 has been marked as a duplicate of this bug. ***