269967 - Firefox 1.0 doesn't open javascript controlled menues anymore

Status: RESOLVED WORKSFORME

(Firefox :: General, defect)

Description

[Thomas Klinger]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; de-DE; rv:1.7) Gecko/20040803 Firefox/0.9.3
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; de-DE; rv:1.7) Gecko/20040803 Firefox/0.9.3

In Firefox 0.9.3 it was possible to use all the functionalities in a Confixx
installation like expanding Javascript-controlled menues. Now in 1.0 this is not
possible anymore even when all extended options in the Javascript menu are
marked. I can see that the cursor changes its look (like for a link) but nothing
happens now. No menu is expanded anymore.



Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Actual Results:  
The menues are not displayed anymore on the web page.

Expected Results:  
They should open in 1.0 as well as in 0.9.3. In the previous 0.9.3 version the
menues were expanded as usual.

Here's some code I fetched from my Confixx:
This is a table containing a cell with the text "Einstellungen". When clicking
on this the below table should be shown.
<table border="0" cellspacing="0" cellpadding="0" width="100%" class="navClosed"
id="menu_block_setings">
			<tr>
				<td>

	
					<table border="0" cellspacing="0" cellpadding="0" width="100%"
class="navTitle" onclick="return opentree ('menu_block_setings');">
					<tr>
						<td class="titleLeft"><img src="/skins/skin_10/images/1x1.gif" width="14"
height="1" border="0" alt=""/></td>
						<td class="titleText" width="100%">Einstellungen</td>
						<td class="titleHandle" id="menu_block_setings_title"><img
src="/skins/skin_10/images/1x1.gif" width="22" height="1" border="0" alt=""/></td>
						<td class="titleRight"><img src="/skins/skin_10/images/1x1.gif" width="3"
height="1" border="0" alt=""/></td>
					</tr>
					</table>

	
	
	
	
				</td>
			</tr>
			<tr>
				<td>
      <div class="tree" id="menu_block_setings_tree">
					<table border="0" cellspacing="0" cellpadding="0" width="100%">
						<tr>
							<td>
 
				 <table border="0" cellspacing="0" cellpadding="0" width="100%"
id="menu_node_setings_autoresponder" class="node">

				 <tr>
			 		<td class="nodeImage" id="menu_item_autoresponder"><a
title="Autoresponder" href="#" onclick="return openScreen
('menu_node_setings_autoresponder', 'email_autoresponder.php',
'ver_xp30_frame_help.php?help_head_key=&help_text_key=',
'ver_xp30_frame_top.php?head_id=autoresponder&head_text=Autoresponder');"><img
src="/skins/skin_10/images/1x1.gif" width="16" height="16" border="0"
alt=""></a></td>
					<td width="100%" style="vertical-align: middle;"><span class="name"><a
title="Autoresponder" href="#" onclick="return openScreen
('menu_node_setings_autoresponder', 'email_autoresponder.php',
'ver_xp30_frame_help.php?help_head_key=&help_text_key=',
'ver_xp30_frame_top.php?head_id=autoresponder&head_text=Autoresponder');">Autoresponder</a></span></td>

				 </tr>
				 </table>
 
				 <table border="0" cellspacing="0" cellpadding="0" width="100%"
id="menu_node_setings_spamfilter" class="node">
				 <tr>
			 		<td class="nodeImage" id="menu_item_spamfilter"><a title="Spam Filter"
href="#" onclick="return openScreen ('menu_node_setings_spamfilter',
'email_spam_config.php',
'ver_xp30_frame_help.php?help_head_key=&help_text_key=',
'ver_xp30_frame_top.php?head_id=spamfilter&head_text=Spam+Filter');"><img
src="/skins/skin_10/images/1x1.gif" width="16" height="16" border="0"
alt=""></a></td>
					<td width="100%" style="vertical-align: middle;"><span class="name"><a
title="Spam Filter" href="#" onclick="return openScreen
('menu_node_setings_spamfilter', 'email_spam_config.php',
'ver_xp30_frame_help.php?help_head_key=&help_text_key=',
'ver_xp30_frame_top.php?head_id=spamfilter&head_text=Spam+Filter');">Spam
Filter</a></span></td> 
				 </tr>

				 </table>
							</td>
						</tr>
					</table>
      </div>
				</td>
			</tr>
		</table>

The Javascript source for:
var treeopened = null;
function opentree(tree) {

        var cls = '';
        if (document.getElementById) {
                var el = document.getElementById (tree);
				var tmpel;
                if (el && el.className) {
                        el.className = (el.className == 'navOpened') ?
'navClosed' : 'navOpened';
						tmpel = document.getElementById( tree + "_tree" );
						tmpel.className = tmpel.className;
						tmpel = document.getElementById( tree + "_title" );
						tmpel.className = tmpel.className;
                }
        }
        if (navigator.appName == 'Microsoft Internet Explorer' &&
document.documentElement && navigator.userAgent.indexOf ('Opera') == -1)
parent.setScrollInIE();
        return false;
}


function openScreen (id, content_url, help_url, top_url) {
	if (activeItem != '' ) {
		var cur = document.getElementById(activeItem);
		cur.className = 'node';
	}
	activeItem = id;
	var el = document.getElementById(id);
	el.className='nodeActive';
	if (parent.contentFrame.frames['mainFrame']) {
	    parent.contentFrame.frames['mainFrame'].location.href = content_url;
	}
	if (parent.contentFrame.frames['mainFrame']) {
	    parent.contentFrame.frames['helpFrame'].location.href = help_url;
	}
	parent.topFrame.location.href = top_url;
//	parent.contentFrame.location.href = url;

	return false;
}

Comment 1

[Thomas Klinger]

One information I forgot:
the openScreen() function works as expected. The trobule maker is opentree().

Comment 2

[José Jeria]

Please attach a minimized testcase that illustrates the bug:
https://bugzilla.mozilla.org/attachment.cgi?bugid=269967&action=enter

Comment 3

[Thomas Klinger]

Attachment: Testcase to Bug 269967

I saved the navigation tree were Firefox 1.0 isn't able to open and close the
menues. 0.9.3 works as expected.
The file attached is a zip-file.

Comment 4

[José Jeria]

Thomas, with minimized testcase I meant a testcase with minimal amount of code
that shows the bug. You added a zip with a bunch of files that most probably are
not needed to show the bug.

Comment 5

[Thomas Klinger]

Attachment: New testcase for 269967

I tried to combine all the functionality into one HTML file. Please excuse the
amount of CSS stuff in there but I didn't developed it so I was not sure what
I've to leave or not (the original code is longer).
Anyway here you see what I meant. When you open the file with Firefox 0.9.3 you
see the "Block item" but the "Menue item" is hidden. Clicking on "Block item"
should expand the table to view also the menue item.
In Firefox 1.0 the expanding didn't work. If it will work NOW then I do not
know why it does not work in the original version (which I sent you but you
discarded it because of too much files).

Comment 6

[José Jeria]

It works the same in Firefox as IE with the last testcase. I get Javascript
errors in both browsers though.

Comment 7

[Thomas Klinger]

It didn't work for me with 1.0 but with 0.9.x as expected. IE I didn't test,
haven't used it since the installation of my PC. :)
I didn't get any errors in the console.

Comment 8

[pbrobinson]

I get a similar problem to this with FF version 1. The site I have problems
with, dare I mention it, is the Microsoft MCP website. If you go to
http://www.microsoft.com/learning/mcp/default.asp there is a pop out menu down
the side which use to work with FireFox (about 0.9 I think) that no longer works
with FF.

The menu entries 'MS Training', 'MS Pres Books', 'MS Certifications' & 'Exams'
all should have sub menus, also in IE they have little right pointing arrows to
inicate this that don't show up in IE.

Comment 9

[José Jeria]

(In reply to comment #8)
> I get a similar problem to this with FF version 1. The site I have problems
> with, dare I mention it, is the Microsoft MCP website. 

That is not what this bug is about.

This is WFM: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2)
Gecko/20050324 Firefox/1.0+

Pleae reopen if you can reproduce with a nightly build:
http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-trunk/