Only allow one instance to be created

RESOLVED FIXED

Status

Other Applications
QA Companion
--
enhancement
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: aakashd, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

10 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20090119 Shiretoko/3.1b3pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20090119 Shiretoko/3.1b3pre

As of now, QAC allows users to run more than one unique instance of the program. Usabilities issues such as confusion due to redundancy and performance of QAC can come up due to this. Also, logically, all users are only really going to be running one test group at a time.

Reproducible: Always

Steps to Reproduce:
1. Go to Tools | QA Companion
2. Focus in back to the browser and go to Tools | QA Companion
Actual Results:  
Multiple QAC instances pop up

Expected Results:  
One QAC instance should be running and if the user clicks on Tools | QA Companion when its already running, the focus should be put back on the already running instance.

Comment 1

10 years ago
Seems more like an enhancement, to request a feature not currently there. Agree?
Severity: normal → enhancement
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → All
Hardware: x86 → All
Summary: QAC allows for more than one instance to be created → Only allow one instance to be created
Version: unspecified → Trunk

Comment 2

10 years ago
Hey Clint. Another QAC bug for you.

Comment 3

10 years ago
It is indeed an enhancement, and one that is needed.  Thanks for CCing me on it, Tyler.
Created attachment 358644 [details] [diff] [review]
possible fix

when you open QAC this will either focus the currently open QAC window or open a new one if you don't have one open already.

Comment 5

10 years ago
Comment on attachment 358644 [details] [diff] [review]
possible fix

>Index: chrome/content/qa.js
>===================================================================
>--- chrome/content/qa.js	(revision 21688)
>+++ chrome/content/qa.js	(working copy)
>@@ -39,8 +39,12 @@
>   htmlNS: "http://www.w3.org/1999/xhtml",
> 
>   openQATool : function() {
>-    window.open("chrome://qa/content/qa.xul", "_blank",
>-                "chrome,all,dialog=no,resizable=yes");
>+    var wm = Components.classes["@mozilla.org/appshell/window-mediator;1"]
>+              .getService(Components.interfaces.nsIWindowMediator);
Align .getService underneath the "C" not the "o". r=ctalbert with that nit.

Thanks for the patch.
Attachment #358644 - Flags: review+
Created attachment 359197 [details] [diff] [review]
patch updated to comment

patch with alignment fix
Attachment #358644 - Attachment is obsolete: true

Comment 7

10 years ago
(In reply to comment #6)
> Created an attachment (id=359197) [details]
> patch updated to comment
> 
> patch with alignment fix
Doh!  I meant to tell you that I'd just do this before I checked it in.  I'm sorry.  Next time it's such a trivial fix, just have me do it before checking in so you don't have to attach a new patch.  Thanks for the patch though, it's very helpful.

Checked in: 
svn commit -m "Bug 474858: Only one instance of QAC window should be opened patch by harthur, r=ctalbert"
Sending        chrome/content/qa.js
Sending        chrome/content/qa.xul
Transmitting file data ..
Committed revision 21842.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.