Download Manager Windows should only flash if already open not honored

RESOLVED FIXED in SeaMonkey2.63

Status

defect
RESOLVED FIXED
7 months ago
7 months ago

People

(Reporter: jou, Assigned: frg)

Tracking

SeaMonkey 2.53 Branch
SeaMonkey2.63
Dependency tree / graph

SeaMonkey Tracking Flags

(seamonkey2.49esr unaffected, seamonkey2.53 affected, seamonkey2.57esr fixed, seamonkey2.60 fixed)

Details

Attachments

(1 attachment)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:56.0) Gecko/20100101 Firefox/56.0 SeaMonkey/2.53

Steps to reproduce:

Start Seamonkey fresh (fresh profile, if you want).
Check Download Setting: Set to "Open download manager" + "Just flash the download manager if it is already open".
Go to any site.
Rightclick picture: download, save.
Download manager opens and is foreground.
Rightclick another picture: download, save.
The already open download manager gets put in foreground.


Actual results:

As for SM 2.49 (and all previous) the download manager only flashes when it is already open, but is not put foreground.
But with SM 2.53 it is put in foreground, so I need extra mouseclicks to activate the main seamonkey window again.


Expected results:

Download Manager should stay in background and only flash.
Exact build Info:
User agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:56.0) Gecko/20100101 Firefox/56.0 SeaMonkey/2.53
Build identifier: 20181210134801
Source: http://www.wg9s.com/
OS: Win10 x64 1809

Current ugly workaround with an autohotkey script:

#NoEnv  ; Recommended for performance and compatibility with future AutoHotkey releases.
;send Seamonkey download Manager to background (seamonkey 2.53 bug workaround)
Loop
{
  WinGet, Active_ID, ID, A
  WinGet, Active_Process, ProcessName, ahk_id %Active_ID%
  WinGetActiveTitle, Title
  ;Only on Seamonkey.exe
  If ( Active_Process ="seamonkey.exe" )
  {
    ;Only if DL-Manager is the active window
    IfInString, Title, Download Manager
    {
      ;Ignore if we actually click the DL-Manager to drag around
      ;Not needed, but prevents moving the DL-Manager as background window.
      if (GetKeyState("Lbutton") = 0)
      {
         SetTitleMatchMode, 2
         WinActivate, - SeaMonkey
      }
    }
  }
  Sleep, 25
}
Confirmed. Let me see if I can do something about it.
Status: UNCONFIRMED → NEW
Depends on: 888915
Ever confirmed: true
The idl file was missing the download parameter for the show function. With nsIDownload gone and this not really using the wrapped js download during Manager startup I opted for a simple bool. This just indicates if a new download is started. All other cases are currently user interaction which should show the manager.

Will be in Bills next builds today or tomorrow too.
Assignee: nobody → frgrahl
Status: NEW → ASSIGNED
Attachment #9031762 - Flags: review?(iann_bugzilla)
Attachment #9031762 - Flags: approval-comm-esr60?
That was quick. Will report success (tough not as fast as you fixed it).
Comment on attachment 9031762 [details] [diff] [review]
1514585-DownloadManagerFlash.patch

a/r=me
Attachment #9031762 - Flags: review?(iann_bugzilla)
Attachment #9031762 - Flags: review+
Attachment #9031762 - Flags: approval-comm-esr60?
Attachment #9031762 - Flags: approval-comm-esr60+
Confirmed: Fix works!
Build identifier: 20181216130004
Works with me too.
So fixed
Status: ASSIGNED → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
bblack keep in mind that a fix in a WG9s build is not a fix here. No push no resolve. And it actually caused Bug 1514729.
Status: REOPENED → ASSIGNED
Blocks: 1514729
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/5725e65df3d5
Fix missing idl parameter to only flash Download Manager when requested via pref. r=IanN
Status: ASSIGNED → RESOLVED
Closed: 7 months ago7 months ago
Resolution: --- → FIXED
Give Bills build a day because of bug 1514729 and you can mark it verified again then :)
Target Milestone: Future → SeaMonkey2.63
You need to log in before you can comment on or make changes to this bug.