Closed Bug 145561 Opened 22 years ago Closed 22 years ago

[OS/2] Focus-stealing JScript improperly processed

Categories

(SeaMonkey :: UI Design, defect)

Product:
SeaMonkey
Component:
UI Design
Platform:
x86
OS/2
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: cowwoc2020, Assigned: mkaply)

References

(
URL
)

Details

Attachments

(1 file)

Complete rewrite of focus on OS/2
12.21 KB, patch
jhpedemonte
: review+
mkaply
: superreview+
jud
: approval+
Details | Diff | Splinter Review 
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.0.0+) Gecko/20020518
BuildID:    2002051808

Google's input field does show a flashing cursor, but it doesn't hold the focus
as it should.

Reproducible: Always
Steps to Reproduce:
1. Visit URL
2.
3.
This was caused by the checkin for:

http://bugzilla.mozilla.org/show_bug.cgi?id=130969
Status: UNCONFIRMED → NEW
Ever confirmed: true
Mine
Assignee: sgehani → mkaply
Complete rewrite of focus as well as ability to debug focus.

I have tested this many different ways and it seems to fix quite a few
problems.

We still have some focus issues with the Java plugin, but I have resigned
myself to the fact that those can't be fixed right now.
*** Bug 72095 has been marked as a duplicate of this bug. ***
*** Bug 140881 has been marked as a duplicate of this bug. ***
*** Bug 145167 has been marked as a duplicate of this bug. ***
*** Bug 138265 has been marked as a duplicate of this bug. ***
Attachment #84682 - Attachment description: Complete rewrite of focus → Complete rewrite of focus on OS/2
Comment on attachment 84682 [details] [diff] [review]
Complete rewrite of focus on OS/2

looks good, r=pedemont
Attachment #84682 - Flags: review+
Comment on attachment 84682 [details] [diff] [review]
Complete rewrite of focus on OS/2

sr=blizzard (platform specific code)
Attachment #84682 - Flags: superreview+
Keywords: mozilla1.0.1
please checkin to the 1.0.1 branch ASAP. once there, remove the mozilla1.0.1+
keyword and add the fixed1.0.1 keyword.
Keywords: mozilla1.0.1 → mozilla1.0.1+
Attachment #84682 - Flags: approval+
Needs adt approval as well.
Keywords: adt1.0.1
OS/2 only code so doesn't need ADT approval
Keywords: adt1.0.1
Summary: [regression] Focus-stealing JScript improperly processed → [OS/2] Focus-stealing JScript improperly processed
Blocks: 140346
fix checked into trunk and branch.

Note trunk checkin was 5/21/2002 accidentally as defect 146252
Status: NEW → RESOLVED
Closed: 22 years ago
Keywords: mozilla1.0.1+ → fixed1.0.1
Resolution: --- → FIXED
Verified. Focus is much better now
Status: RESOLVED → VERIFIED
Product: Core → Mozilla Application Suite
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: