261186 - Popup blocker disabled but popup JavaScript link does not work

Status: RESOLVED DUPLICATE of bug 194404

(Firefox :: General, defect)

Description

[Nadir Elzein]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; rv:1.7.3) Gecko/20040914 Firefox/0.10
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; rv:1.7.3) Gecko/20040914 Firefox/0.10

This is an actual JavaScript function with a popup that does not work:

function OpenTimeSheetProfilePopup(tsid)
{

	if (ynDirty == 1)
	{
		alert('You have made changes to one or more line items.  Please cancel these
changes or save them by pressing the \'Save Time Entries\' button before
proceeding.');
		return;
	}

	var url = urlEncode("/trsv2/popups/TimeSheetProfile.aspx?tsid=" + tsid);

	var h          = GetCookie("timesheetprofileHeight");
	var w          = GetCookie("timesheetprofileWidth");
	
	var returnVal;
	if(h == null || w == null )
		returnVal = showModalDialog("/trsv2/FrameWrapper.aspx?url=" + url, window,
"dialogHeight:700px;dialogWidth:698px;center:yes;resizable:yes;scroll:no")
	else
		returnVal = showModalDialog("/trsv2/FrameWrapper.aspx?url=" + url, window,
"dialogHeight:" + h + "px;dialogWidth:" + w +
"px;center:yes;resizable:yes;scroll:no");

	submit = document.getElementById('TimeEntry:btnCancel');
	submit.click();	

}

Reproducible: Always
Steps to Reproduce:
1. Visit http://www.palmone.com/us/support/downloads/windesk414_legal.html
2. Click on "German PalmDesktopWin414DE.zip
3.

Actual Results:  
Popup does not come up and the same existing page just refreshes.

Expected Results:  
Popup new window should come up

Comment 1

[Nadir Elzein]

Actual URL to reproduce is behind firewall and not accessible from outside. I
could not find a similar URL that reproduces the problem, but I've provided the
actual JavaScript code on which the popup fails.

Comment 2

[Jesse Ruderman]

showModalDialog() is IE-specific.  If you look at the JavaScript console after
clicking that link, you'll probably see something like "showModalDialog is not a
function". Use window.open() instead.

Comment 3

[Dan M]

*** This bug has been marked as a duplicate of 194404 ***