125101 - bovision.se - bad show/hide layer code

Status: RESOLVED FIXED

(Tech Evangelism Graveyard :: Other, defect)

Description

[Ping Liao]

The 'talk bubbles' on the page that should only occur when you move the mouse
over the corresponding link. Now they are always present which therefore hides
whatever below them.

This website works fine with NS 4.7x and IE 6.x, but does not work correctly
on NS 6.2 Windows and Solaris.

This problem is reported by Christer Jansson <[e-mail]>

Comment 1

[Jim Crumley]

Confirming on Sparc Solaris build 2001013122.

This might be evangelism as well, but I don't know enough javascript to be able
to tell.

Comment 2

[John Levon]

same as bug 119436 I think. This one has a test case though...

Comment 3

[timeless]

definitely not JSEng

Comment 4

[R.B.]

The 'talk bubbles' in the URL are controlled by the JS function:

function showHide(obj,sStatus)
{
	if (document.layers) //Netscape
	{
		if (sStatus == "show")
			eval("document."+obj).visibility = "show";
		else
			eval("document."+obj).visibility = "hide";
	}
	else //IE
	{
		if (sStatus == "show")
			eval(obj).style.visibility = "visible";
		else
			eval(obj).style.visibility = "hidden";
	}
}

which is called like:

onMouseOver="showHide('inredning','show');"
onMouseOut="showHide('inredning','hide');"

'inredning' is <div> Id. I don't think this should work, over to evangelism ?

Comment 5

[Bob Clary [:bc] (inactive)]

Yes, we will take. -> Euro West

Comment 6

[Mathieu Pillard [:mat]]

looking at the site I dont see anything wrong. seems FIXED to me now.

Comment 7

[Mathieu Pillard [:mat]]

mmm, it works now but they use ua detection instead of object detection, so it
might not work with every gecko browser.

heres a working version of the showhide function that uses object detection :

function showHide(obj,sStatus)
{
	if (document.layers) //Netscape 4
	{
		if (sStatus == "show")
			eval("document."+obj).visibility = "show";
		else
			eval("document."+obj).visibility = "hide";
	}
	else if (document.all) //IE
	{
		if (sStatus == "show")
			eval(obj).style.visibility = "visible";
		else
			eval(obj).style.visibility = "hidden";
	}
        else if (document.getElementById)
	{
		if (sStatus == "show")
			document.getElementById.style.visibility = "visible";
		else
			document.getElementById.style.visibility = "hidden";
	}
}

Comment 8

[Frederic Bezies]

My swedish is very poor :-/

Contact (possible) : [e-mail]

Comment 9

[Mathieu Pillard [:mat]]

can someone contact them with the fix ?

Comment 10

[Stefan [:stefanh]]

Some contacts:
[e-mail] -responsible for the site.
[e-mail] - technical resp
[e-mail] - technical development

Frederic: Swedes speak well English

Comment 11

[Kai Lahmann (is there, where MNG is)]

move...

Comment 12

[Bob Clary [:bc] (inactive)]

other default owner

Comment 13

[Marek Stępień [:marcoos, inactive]]

Site now works ok, the script has been corrected. Checked this with current
Firefox/aviary and Netscape 7.1.

Fixed.