Bug 511293

Summary:	url on /usr/bin/seamonkey command line forced to new window
Product:	[Fedora] Fedora	Reporter:	Tom Shield <twshield>
Component:	seamonkey	Assignee:	Kai Engert (:kaie) (inactive account) <kengert>
Status:	CLOSED WONTFIX	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	low	Docs Contact:
Priority:	low
Version:	11	CC:	caillon, gecko-bugs-nobody, kengert
Target Milestone:	---	Keywords:	EasyFix, Patch
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2009-07-22 21:29:20 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Tom Shield 2009-07-14 16:05:07 UTC

Description of problem:

/usr/bin/seamonkey script forces urls to be opened in new window instead of letting user preferences decide.

Version-Release number of selected component (if applicable):

seamonkey-1.1.16-1.fc11.i586.rpm

How reproducible:

every time


Steps to Reproduce:
1. open seamonkey
2. in preferences for tabbed browsing select new url's open in new tabs in current window
3. /usr/bin/seamonkey http://cnn.com
  
Actual results:

a new seamonkey window opens

Expected results:

a new tab in the current seamonkey window opens.

Additional info:

Fix to /usr/bin/seamonkey is change the 3rd from last line of the script from

exec $MOZ_CLIENT_PROGRAM "openurl($opt,new-window)" 2>/dev/null >/dev/null

to

exec $MOZ_CLIENT_PROGRAM "openurl($opt)" 2>/dev/null >/dev/null

Comment 1 Kai Engert (:kaie) (inactive account) 2009-07-22 21:29:20 UTC

This isn't a bug, but a feature request.
I disagree to make this change.

"Open in a new window" is a safe operation for everyone.

With your proposed change, if some other application starts seamonkey, the user may lose the work in the current window.

A better change might be to improve the seamonkey start script to accept another parameter.

Comment 2 Kai Engert (:kaie) (inactive account) 2009-07-22 21:30:10 UTC

If you produce a patch that adds such a new parameter, or if you disagree with my words, you may reopen this bug or file a new one (for the new feature).
Thanks.

Comment 3 Tom Shield 2009-07-23 15:06:09 UTC

The current behavior of your /usr/bin/seamonkey script is different from the behavior you get by calling the /usr/local/seamonkey/seamonkey script that comes with the seamonkey stock package.  I got the behavior I wanted (determined by my seamonkey preferences) when I used that script.  It seems to me the whole point of your /usr/bin/seamonkey script is to provide a static location to provide this behavior because you put seamonkey in version specific directories.

If a new URL is opened in a window you are already using the back button will get you back to your original URL, nothing is lost.

Adding this behavior as another parameter to the script, instead of default behavior, would also break compatibility the stock script.  I'm using this script from alpine to display links/attachments, etc, and it expects the stock behavior, as, I expect, would most apps that call this.