221557 - maxhavelaar.nl - Links don't work due javascript problem

Status: RESOLVED FIXED

(Tech Evangelism Graveyard :: Dutch, defect, P3)

Description

[Onno]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20030927
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5) Gecko/20030927

Try to push on any link, nothing will happen. The link has a form of
javascript:goTo('nieuws_detail.asp?nGroepID=2', '&rID=20&nieuwsID=45&hp=Y',
'#EEB40E');

I don't know if this is due Microsoft standards.

Reproducible: Always

Steps to Reproduce:
1. try pussing link
2.
3.

Actual Results:  
nothing happens

Expected Results:  
go to url

Comment 1

[Olivier Cahagne]

JS console shows:

Error: parent.frmMenu.bgChanger has no properties
Source File: http://www.maxhavelaar.nl/js/functions.inc.js
Line: 3

Comment 2

[Olivier Cahagne]

source shows:

function goTo(strTemplate, strVars, strColor) {
	if(strTemplate == '') strTemplate = 'template.asp';
	parent.frmMenu.bgChanger.style.backgroundColor = strColor;
	parent.frmContent.document.location.href = strTemplate + strVars;
}

I think bgChanger is IE-only. Can someone confirm and move to Tech Evangelism ?

contact: [e-mail]

Comment 3

[Patrick]

http://www.a1javascripts.com/mouseover_scripts/exbutton/exbutton.html says that
it  is IE only. 

Moving to TechEvang, reassigning and setting to P3.

Comment 4

[Reinout van Schouwen]

I've sent a letter.

Comment 5

[Onno]

Its a real shame that those CSS menu's don't work. Maxhavelaar is for fair world
(North/south) trade and fair cofee. Its like a comercial NGO. So they have all
reasons to be W3C compliant.

A lot of NGO persons come eventualy on this site. 

Comment 6

[Chris Hayes]

Not knowing about this bugreport, I also ran into this and 
mailed the site devs to solve it. Apart from a 'we will look into it' in April
they never responded. And I am even SELLING their products.


Solution?
They can easily solve it by adding 

if (IE) 

before the line causing the error, as they already have an IE test in the same
js file. 
Will solve the 'lees meer' (read more) problem as well as the dropdown menus.

chayesATantennaDOTnl

Comment 7

[Tim Maks van den Broek [:mad_maks]]

The links on the site are working fine now, so can we close this bug?

Comment 8

[Patrick]

Jepp, site looks OK now --> fixed