89979 - buy.com - Submenus provided only for IE, not Mozilla/Netscape

Status: RESOLVED FIXED

(Tech Evangelism Graveyard :: English US, defect, P1)

Description

[Phil Schwartau]

Site provides submenus only for browsers that support document.all.
Here are two included JS files used at the site, showing the problem:

http://www.us.buy.com/retail/menu_sub.js :

//* Netscape check - DO NOT LET NS IN
var isMac = navigator.appVersion.indexOf("Mac")>-1
if(document.all && !isMac)
{
document.write ("<table ...   

        etc.  
        etc.



Similarly, this included JS file defines functions used in 
mouseovers at the site, and it similarly excludes non-IE browsers
http://www.us.buy.com/retail/menu.js 

// Build menu
function getMenu(deptName,menuName) {                
 if(document.all && !mac) {             
   var top = 0;
   var left = 0;
   var thisDept;
                        
   if(vizDept != null && vizMenu != null) {


             etc.
             etc.


For example, here is the mouseover definition for the "Computers" 
link at the site:

   <td id="computers" onmouseover="getMenu('computers', 'ComputerMenu')">

Because of the non-standard JavaScript, you will get a submenu 
under it in IE, but not in Moz/NN ...

Comment 1

[Phil Schwartau]

Priority --> P1 because of the commercial importance of this site.

Comment 2

[John Unruh]

*** Bug 89980 has been marked as a duplicate of this bug. ***

Comment 3

[John Unruh]

*** Bug 89981 has been marked as a duplicate of this bug. ***

Comment 4

[Bob Clary [:bc] (inactive)]

Can't find a technical contact for the site. Sending webmaster letter to
[e-mail]

Comment 5

[Phil Schwartau]

Compare Bugscape bug:
 
                  http://bugscape/show_bug.cgi?id=5110

Comment 6

[Bob Clary [:bc] (inactive)]

All Evangelism Bugs are now in the Product Tech Evangelism. See bug 86997 for
details.

Comment 7

[Bob Clary [:bc] (inactive)]

follow up with these sites by 0.9.5

Comment 8

[Bob Clary [:bc] (inactive)]

-> 0.9.6, need to follow up on these asap

Comment 9

[Doron Rosenberg (IBM)]

still not fixed, resetting bug

Comment 10

[Christopher Aillon (sabbatical, not receiving bugmail)]

Moving.  if buy.com ain't Ecommerce, I don't know what is :)

Comment 11

[Doron Rosenberg (IBM)]

Some other interesting code:

if ((navigator.userAgent.indexOf("Netscape6/6.1") != -1) ||
(window.location.hostname.indexOf("netsagecentral.com") != -1) || mac) {
			displaySage = false

Comment 12

[d.cary]

This isn't just about optional eyecandy (no-click fly-down menus), it's about
basic content. The buy.com site seems to be actively rejecting potential
customers who use the ``wrong'' web browser.

I can't even use Mozilla to log in
  https://www.us.buy.com/corp/support/login.asp
.

I think the culprit is this code:

  <form [...snip...]>
  [... lots of snippage ...]
  <a href="javascript:document.MoreTrackingInfo.submit()">
  <input type="image" src="https://....../btn_text_login2.gif" >
  </a>
  [...snip...]
  </form>

If I understand
  http://www.w3.org/TR/REC-html40/interact/forms.html
correctly, it would be simpler to do something like this:

  <form [...snip...]>
  [... lots of snippage ...]
  <BUTTON name="submit" value="submit" type="submit">
  <IMG src="https://....../btn_text_login2.gif" alt="login"></BUTTON>
  <input type="submit" value="Login">
  </form>

(Mozilla can handle this, right ?).

Comment 13

[Doron Rosenberg (IBM)]

http://www.nexgenmedia.net/evang/buy.com/default.asp.html

Fixed code, menu.js and menu_sub.js.

Anyone want to contact them?

Comment 14

[Doron Rosenberg (IBM)]

marking fixed, no dhtml menus being used anymore in IE as well.

Comment 15

[Bob Clary [:bc] (inactive)]

tech evang june 2003 reorg