Closed Bug 200656 Opened 22 years ago Closed 22 years ago

MailAreaHasFocus() is no longer necessary

Categories

(SeaMonkey :: MailNews: Message Display, defect)

x86
Windows 95
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.6alpha

People

(Reporter: neil, Assigned: sspitzer)

Details

(Keywords: perf)

Attachments

(2 files)

Possibly due to some recent key event code changes, <input>s and <textbox>es now prevent character <key>s from firing thus making the check redundant.
Attached patch Proposed patchSplinter Review
The other advantage is that textboxes don't disable the menu items any more.
Attachment #119441 - Flags: superreview?(sspitzer)
Attachment #119441 - Flags: review?(cavin)
Comment on attachment 119441 [details] [diff] [review] Proposed patch Changing some out-of-date reviewers...
Attachment #119441 - Flags: superreview?(sspitzer)
Attachment #119441 - Flags: superreview?(bienvenu)
Attachment #119441 - Flags: review?(scott)
Attachment #119441 - Flags: review?(cavin)
Attachment #119441 - Flags: review?(scott) → review+
Comment on attachment 119441 [details] [diff] [review] Proposed patch great, thx, sr=bienvenu
Attachment #119441 - Flags: superreview?(bienvenu) → superreview+
/me must remember to check these in :-)
Target Milestone: --- → mozilla1.6alpha
For completeness, this is the patch as checked in.
Fix checked in.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: