Closed
Bug 98657
Opened 23 years ago
Closed 23 years ago
remove <commandset id="composerSaveMenuItems"/> from messengercompose.xul
Categories
(MailNews Core :: Composition, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: sspitzer, Assigned: sspitzer)
Details
Attachments
(1 file)
6.81 KB,
text/plain
|
Details |
remove <commandset id="composerSaveMenuItems"/> from messengercompose.xul having it causes us to overlay a some editor command stuff that we don't care about. (see http://lxr.mozilla.org/mozilla/source/editor/ui/composer/content/editorOverlay.xul#122) specificially, we do some command updating on the create and save events. we've got File | Save (and a save button), but I'm pretty sure none of it is hooked up to how editor does save, we do save all on our own. obviously, before you remove it should should confirm that I'm right that we don't need it. this won't be a huge performance win, but we should still get to it at some point.
Updated•23 years ago
|
QA Contact: sheelar → stephend
Assignee | ||
Comment 1•23 years ago
|
||
taking, I've got this in my tree. not a big win, but still something.
Assignee: varada → sspitzer
Assignee | ||
Comment 2•23 years ago
|
||
fixed
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Ignore the typo (10-16 is actually 10-26). Verified FIXED, based on the logs
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•