Closed
Bug 159729
Opened 22 years ago
Closed 22 years ago
-turbo shouldn't steal focus
Categories
(SeaMonkey :: UI Design, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: cowwoc2020, Assigned: mkaply)
Details
Attachments
(2 files)
517 bytes,
patch
|
jhpedemonte
:
review+
jhpedemonte
:
superreview+
|
Details | Diff | Splinter Review |
57.57 KB,
patch
|
jhpedemonte
:
review+
jhpedemonte
:
superreview+
|
Details | Diff | Splinter Review |
From Bugzilla Helper: User-Agent: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.1b) Gecko/20020724 BuildID: 2002072408 Running Mozilla -turbo (seemingly) causes it to open up and minimize very quickly to accomplish its goal. You shouldn't be poping up anything at all. If I place "mozilla -turbo" in my startup script it keeps on stealing the focus away from me while I'm typing in some other window. Mozilla -turbo should run completely transparent to the end-user and should not involve any visible GUI components. Reproducible: Didn't try
Reporter | ||
Comment 1•22 years ago
|
||
Correction to title :) changed "should steal focus" to "shouldn't steal focus"
Summary: -turbo should steal focus → -turbo shouldn't steal focus
Assignee | ||
Comment 2•22 years ago
|
||
Taking
Assignee: sgehani → mkaply
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 3•22 years ago
|
||
There are really two problems here. Problem 1 is that the window comes up at all. Problem 2 is the focus. This first patch is the focus issue. The second patch will be a rewrite of nsNativeAppSupportOS2.cpp to bring our turbo up to Windows and hence fix the rest of the problems.
Assignee | ||
Comment 4•22 years ago
|
||
Every so often we have to report nsNativeAppSupport to get the latest stuff from windows, especially for turbo mode and DDE. That's what this is. This fixes some misc traps with turbo mode, as well as some DDE functions that weren't working. It's also some general cleanup of the file to be more like Windows.
Comment 5•22 years ago
|
||
Comment on attachment 93454 [details] [diff] [review] Part 1 of fix r=pedemont, sr=blizzard (platform specific code)
Attachment #93454 -
Flags: superreview+
Attachment #93454 -
Flags: review+
Comment 6•22 years ago
|
||
Comment on attachment 93455 [details] [diff] [review] Report of nsNativeAppSupportOS2 if you set MOZ_DEBUG_DDE to 0 instead of one, then: r=pedemont, sr=blizzard (platform specific code)
Attachment #93455 -
Flags: superreview+
Attachment #93455 -
Flags: review+
Assignee | ||
Comment 7•22 years ago
|
||
Fixes checked in.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: Core → Mozilla Application Suite
You need to log in
before you can comment on or make changes to this bug.
Description
•