22704 - [LAYER] Collapse and Decollapse does not work

Status: VERIFIED FIXED

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

Description

[André Langhorst]

Try to click on [+], normally it should decollapse,
IE and even Netscape 4.7 can do it

Comment 1

[Sean Richardson]

Confirmed that the demo at the URL does not work in a current build.
The [+] toggles the expansion/collapse of a list via javascript in NN 4.7.

This may be a DOM Level 1 bug, or the page may be using javascript manipulating
proprietary DOM elements - not sure.  Changing Component from "Javascript
Engine" to "DOM level 1"

Changed URL to point to the left frame of original demo as that is the testcase;
the original is at http://www.phpwebdev.com/phpMyAdmin-demo/

Tested with:
1999-12-25-08-M13 nightly binary on Windows NT 4.0sp3
NN 4.7 on Windows NT 4.0sp3

Comment 2

[John Morrison]

The onclick event is handled by expandIt(arg), which uses roughly this logic:

   NS4=(document.layers) ? true : false;
   IE4=((document.all)&&(bV>=4))?true:false;
   ver4 = (NS4 || IE4) ? true : false;

   function expandIt(el) {
       if (!ver4) return;
       if (IE4) {expandIE(el)} else {expandNS(el)}
   }

So, the script will not work with Mozilla under any circumstance
(i.e., 'document.layers == null' and 'document.all == undefined' on
Mozilla/5.0)

So, marking this bug [LAYER] and INVALID (but I (or someone) should really
send a note to the php people to use W3C DOM, and proper user-agent sniffing).

Comment 3

[Robert Ginda]

A similar issue was reported in bug 22517

Comment 4

[Prashant Desale]

[LAYER] Bug. Verified.

Comment 5

[ekrock's old account (dead)]

Moving all [LAYER] bugs to Evangelism component for tracking and open-source
evangelism by mozilla community members of sites that need to upgrade to support 
web standards such as HTML 4.0 (instead of LAYER/ILAYER) and the W3C DOM
(instead of Nav4 document.layers[] or IE document.all()). Sites should be
lobbied to do the upgrade using the email templates that are linked to from
http://www.mozilla.org/newlayout/bugathon.html#layerbugs . When a site's owner
has confirmed receipt of the message requesting an upgrade, the bug should be
marked with the keyword evangelized to indicate that evangelism for that bug is
complete. When the site finishes the upgrade and supports standards, the bug
should be closed.

Comment 6

[ekrock's old account (dead)]

Closing all Evangelism bugs where no evangelism is needed because page has been 
fixed, site is internal to Netscape, report is a DUP, or bug report is no longer 
appropriate for evangelism for any other reason.

Comment 7

[Zach Lipton [:zach]]

SPAM:Changing QA contact on 111 evang bugs as I am now the new QA contact for 
this component.

Sorry about the spam

zach

Comment 8

[Blake Ross]

Reassigning Evangelism bugs to me, the component's new owner.  I would like to 
take this opportunity to thank [e-mail] for all of his dedication, 
contributions, and hard work, and wish him luck at his new job.  Thanks, nobody.

Comment 9

[Blake Ross]

workaround bugzilla problem that caused a bunch of evangelism bugs to be 
NEW/INVALID, NEW/FIXED, NEW/WORKSFORME or NEW/DUPLICATE

Comment 10

[Blake Ross]

-> [e-mail] for my evangelism bugs.

removing the now-depreciated evangelism-related keywords.

setting platform to All.

Comment 11

[André Langhorst]

this is fixed now, but another bug appears, unfortunately their test page does 
not work, I will file a new bug+testcase
marking this closed then (phpmyadmin has been changed to work with mozilla)

Comment 12

[André Langhorst]

verified then

Comment 13

[Bob Clary [:bc] (inactive)]

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