Closed Bug 107925 Opened 24 years ago Closed 24 years ago

context menu items (cut, copy, paste, delete, select all) don't work

Categories

(SeaMonkey :: Composer, defect)

Product:
SeaMonkey
Component:
Composer
Platform:
x86
Windows 2000
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla0.9.7

People

(Reporter: skasinathan, Assigned: cmanske)

Details

(Whiteboard: EDITORBASE)

Attachments

(1 file)

Fix: Don't use "obserses" for Commands not registered on Composer's content window.
2.40 KB, patch
Brade
: review+
hewitt
: superreview+
Details | Diff | Splinter Review
Right click in the body of the composer. Select paste. Nothing is pasted. I'm not sure abt other menuitems. today's trunk on win2k.
Changed component from Compositor to Editor:Composer
Assignee: kmcclusk → syd
Component: Compositor → Editor: Composer
QA Contact: petersen → sujay
Target Milestone: --- → M1
Target Milestone: M1 → Future
Badness -- duplicated on Windows NT 2001120703. The "Paste" menu item is disabled in menubar, even though the text I copied made it into the clipboard (verified I could paste elsewhere in the app).
Assignee: syd → cmanske
Whiteboard: EDITORBASE
I don't see the "Paste" menuitem disabled, but what I do see is that none of the Cut/Copy/Paste commands work from the popup menu! They do work from the top-level "Edit" menu.
Status: NEW → ASSIGNED
Target Milestone: Future → mozilla0.9.8
The menuitems in the top-level "Edit" menu are overlayed from utilityOverlay.xul, and they call the commands using "goDoCommand()" rather than "observes", so this fixes the context menu commands.
This is important, imho. Fix is ready, so will try to get in for 0.9.7
Keywords: patch, review
Summary: context menu paste doesn't work in the body. → context menu items (cut, copy, paste, delete, select all) don't work
Whiteboard: EDITORBASE → EDITORBASE, FIX IN HAND, need r=,sr=
Target Milestone: mozilla0.9.8 → mozilla0.9.7
Comment on attachment 61169 [details] [diff] [review] Fix: Don't use "obserses" for Commands not registered on Composer's content window. r=brade
Attachment #61169 - Flags: review+
Whiteboard: EDITORBASE, FIX IN HAND, need r=,sr= → EDITORBASE, FIX IN HAND, need sr=
Attachment #61169 - Flags: superreview+
Comment on attachment 61169 [details] [diff] [review] Fix: Don't use "obserses" for Commands not registered on Composer's content window. sr=hewitt
Whiteboard: EDITORBASE, FIX IN HAND, need sr= → EDITORBASE, FIX IN HAND,reviewed
checked in
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Keywords: patch, review
Resolution: --- → FIXED
Whiteboard: EDITORBASE, FIX IN HAND,reviewed → EDITORBASE
looks like several bugs like bug 107635, bug should be fixed as well as other copy/paste bugs in bugzilla with this, yes?
I meant to add bug 79880 to that last comment.
maybe as this is related to composer, does mail/news context menus use the same code path, or is it seperate? and would this fix be the same for mail/news components like bug 107635, and bug 79880.
Mail-News, including Mail Composer, implement their own context menus, but I hope what I found here will help!
This one is fixed. Verified on Win 2k in build 2001121803.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: