176391 - disneygeek.com - menu bar displays slightly wrong

Status: RESOLVED INCOMPLETE

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

Description

[Carl]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2b; MultiZilla v1.1.25) Gecko/20021016
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2b; MultiZilla v1.1.25) Gecko/20021016

In the page http://www.disneygeek.com, the top menubar is supposed to be a
continuous grey area. Instead, mozilla displays little boxes at the lower right
corner of each menu entry.

There is also an extra cell to the left of the menu bar that does not show up in
other browsers.

It may have something to do with the way Mozilla handles Javascript.

Reproducible: Always

Steps to Reproduce:
1.Open mozilla
2.Visit http://www.disneygeek.com

Actual Results:  
The menu bar part of the webpage is not properly displayed. It has an extra cell
to the left and each menu entry has a little rectangle to the lower right.

Expected Results:  
The menu bar should just be continuous gray without the little rectangles at the
bottom right. Also, there shouldn't be an extra cell to the left.

Throughout the rest of this site too, the menu bar is always displaced from it's
intended position.

Comment 1

[Boris Zbarsky [:bzbarsky]]

The little rectangles are the broken images the site has there... that part of
the rendering is completely correct.

Comment 2

[Carl]

After a little tinkering, on a hunch I was able to pull up the site's javascript
code - http://www.disneygeek.com/menu.js

However, I can't see anything wrong. Any ideas anyone?

Comment 3

[Kenneth Herron]

The little boxes on the home page are easy enough to explain. menu.js contains a
method called MemberSetUp(), which contains the following code to construct the
menu labels, e.g. "Disneyland Resort" (I've wrapped the code using indent, which
produces slightly odd results on javascript):

        if (MemVal.indexOf ('<img') == -1 && eval (WhatMenu + '[2]'))
          {
            /* Commented Out by Jason
               S=(CreateRecursLevel==1&&FirstLineHorizontal)?
                 '/images/spacer.gif':'/images/spacer.gif';
             */
            W = (CreateRecursLevel == 1 && FirstLineHorizontal) ? 10 : 5;
            H = (CreateRecursLevel == 1 && FirstLineHorizontal) ? 5 : 10;
            T = (CreateRecursLevel == 1
                 && FirstLineHorizontal) ? Hght - 7 : Hght / 2 - 5;
            L = (CreateRecursLevel == 1
                 && FirstLineHorizontal) ? Wdth - 12 : Wdth - 7;
            if (DomYes)
              {
                t = Location.document.createElement ('img');
                this.appendChild (t);
                t.style.position = 'absolute';

                /*    Commented out by Jason

                   t.src=S;
                 */

        t.style.width = W;
        t.style.height = H;
        t.style.top = T;
      t.style.left = L}
    else
      {
        MemVal +=
          "<div style='position:absolute; top:" + T + "; left:" + L +
          "; width:" + W + "; height:" + H +
          ";visibility:inherit'><img src='" + S + "'></div>";
      this.innerHTML = MemVal}
  }

Here, MemVal is a menu label (e.g. "Disneyland Resorts"). When the label text is
just plain text, the menu label is constructed using the plain text plus a
spacer GIF.  Except that the code to set the spacer's URL is commented out.

I don't see any extra box on the homepage menu. I do see menus on other parts of
the site being positioned incorrectly, but I'm not going to debug this site's
javascript any further.

Comment 4

[Frankie]

At least the menu works.

In MSIE 5.2.2 for Mac the menu displays inside the "What's New" frame, and in
Safari the menu doesn't display at all.

Comment 5

[Bob Clary [:bc] (inactive)]

tech evang june 2003 reorg

Comment 6

[Chris Lawson (gone)]

INCOMPLETE due to lack of activity since the end of 2009.

If someone is willing to investigate the issues raised in this bug to determine whether they still exist, *and* work with the site in question to fix any existing issues, please feel free to re-open and assign to yourself.

Sorry for the bugspam; filter on "NO MORE PRE-2010 TE BUGS" to remove.