171241 - televideo.rai.it - using proprietary JS

Status: RESOLVED INVALID

(Tech Evangelism Graveyard :: Italian, defect, P4)

Description

[Maurilio Venturini]

loading page
http://www.televideo.rai.it/nazionale/homenaz.asp?tlv=280&sottop=00&x=13&y=8

The box "NAVIGA IN TELEVIDEO" remains blank instead of showing the picture
280_00.jpg (as you can see in IE).

Comment 1

[Jim Dunn]

This appears to be a website issue:

They are checking for browser name via:
	var isNav4 = (window.navigator.appName == "Netscape");
and mozilla & its derivatives (ns6/7) get flagged by this.
and so they incorrectly think mozilla is nav4 and then use layers

<SCRIPT LANGUAGE="JavaScript">
if (isNav4) {
        document.layers["tlvNav"].visibility='show';
} else {
        eval('document.all.tlvNav.style.visibility="visible"');
}
</SCRIPT>

the site should be changed to check for mozilla .vs. 4.x .vs. ie
as in http://www.mozilla.org/docs/web-developer/sniffer/browser_type.html

Comment 2

[Jim Dunn]

fyi appName is hardcoded as Netscape in routine
NavigatorImpl::GetAppName(nsAString& aAppName)
around line 5470 in nsGlobalWindow.cpp
http://lxr.mozilla.org/seamonkey/source/dom/src/base/nsGlobalWindow.cpp#5470

Comment 3

[Jim Dunn]

Re-assigning to evangelism.  If they change their "isNav4" check to
the following, I think things will work.

var isNav4 = (window.navigator.appName == "Netscape") &&
(parseInt(window.navigator.appVersion) <= 4);

Comment 4

[Jim Dunn]

reassinging to susie

Comment 5

[Bill Mason]

Confirming new.
Reducing priority.  Critical is not used for TE bugs.

Comment 6

[Nick Bebout]

Moving to new component

Comment 7

[Pascal Chevrel (relman team) -> :pascalc]

initial problem is fixed, but now their scroller doesn't work and flash anims
are not displayed.

Comment 8

[chris hofmann]

looks like the site might be working ok now

Comment 9

[Mike Taylor [:miketaylr]]

That link 404s now.