Closed
Bug 46466
Opened 24 years ago
Closed 22 years ago
XUL for edit menu doesn't list accelerator for "Delete"
Categories
(SeaMonkey :: MailNews: Message Display, defect, P3)
Tracking
(Not tracked)
VERIFIED
WORKSFORME
People
(Reporter: dmosedale, Assigned: sspitzer)
References
(Depends on 1 open bug)
Details
(Keywords: polish)
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
<Del> (ie the delete key) is used as a keyboard accelerator for "Delete
Message". However, the XUL for the dropdown menu doesn't document this.
My bug. But. sfraser: You guys are abusing JavaScript, After nsbeta2 we can
talk about cleanup.
This needs a code review on win, lin, mac and maybe os/2. I also want to know
if what I did in the .xul file is correct. I think I should not have needed to
change the Entity line but I couldn't seem to coax it.
I really want this in before nsbeta2 branches and if it's reviewed it should be
safe.
Assignee | ||
Comment 5•24 years ago
|
||
timeless, feel free to re-assign to me if you want me to take your patch and run
with it.
poof [i didn't rememeber having a patch here]
Assignee: timeless → sspitzer
Status: ASSIGNED → NEW
Comment 8•24 years ago
|
||
That's not the right fix. This will really be fixed by bug 47426, which already
has the correct patch. Marking dependency to make sure this gets fixed when
that does.
Depends on: 47426
Comment 9•24 years ago
|
||
Marking nsbeta1 since "Del" is a common accelerator that should be listed.
Keywords: nsbeta1
QA Contact: lchiang → nbaca
Comment 10•24 years ago
|
||
Fixed the xp menu code, saari has bug 62030 for the Mac menu code, marking this
depend on that.
Depends on: 62030
Comment 11•24 years ago
|
||
marking nsbeta1- but if we a patch for this already then of course it would be
great to get it in.
Comment 12•24 years ago
|
||
Worksforme on Windows98, build 20010306!
Comment 13•24 years ago
|
||
Looks like it's still a problem on the Mac (bug# 62030).
Updated•23 years ago
|
QA Contact: nbaca → olgam
Comment 14•22 years ago
|
||
I put nsbeta1 - seems WFM - requires verification on recent trunk build.
Keywords: nsbeta1
Comment 15•22 years ago
|
||
yep, wfm using 1/3 trunk bits on linux rh8 and win2k.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → WORKSFORME
Updated•20 years ago
|
Product: Browser → Seamonkey
Updated•19 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•