Does not expand/contract menues with Bluemarine Theme [#311886]

Hi People,
I use the built-in Bluemarine Theme as administration theme, because it is quite "visually clean" theme to use in the backend, but DHTML Menu 6.x-2.2 does not work at all. In IE it throws this error:

***********************
Line:32
Char:9
Code:0
Error:El objeto no acepta esta propiedad o método (the object does not accept this property or method)
URL:http://nnn.nnn.nnn.nnn/drupal6/admin/settings/admin
************************

If i change the theme to Garland (another default theme) if works ok. Also using Marinelli, a custom theme it works.

Hope you may fix this because Bluemarine theme is VITAL for me to use in backend, because of its compact and simple interface.

Sorry my not-good-enough english hehe, i'm from Argentina (South America).

See ya, and keep up your great work!!!

Claudio Abraham
(mrbrowny)

Comment	File	Size	Author
#6	dhtml_menu_6_2_ie_sucks-311886-6.patch	2.87 KB	cburschka

Comments

Comment #1

mrbrowny commented 24 September 2008 at 19:57

Priority:

Normal

» Critical

Ok i changed the priority to CRITICAL, because i had to disable this useful module because of using Bluemarine Built-in Theme.

Yes, I may change the administration theme meanwhile, but it's so clear and useful, that meanwhile this is fixed i prefer to disable DHTML Menu module rather than changing the theme.

Comment #2

roald commented 8 October 2008 at 12:54

I also get the same problem in IE6 and IE7 with Bluemarine and Twilight. With Garland it works ok. Also if using Firefox or Opera on all those themes everything is OK

Comment #3

pauline_perren commented 20 October 2008 at 02:42

I was having this problem too, with my Marinelli sub-theme. I discovered that IE7 reports the error on the wrong line, it is actually the line above the one reported, in my case this was line 31 of dhtml_menu.js submenu = document.getElementById(cookieList[i]);

The problem was caused by an apparent reuse of the submenu variable caused by a div with that id in the theme's page.tpl.php file (wrapping around the output of the links for the secondary menu). I have fixed the problem for my theme by changing the name of that div in my sub-theme.

For the bluemarine theme I suspect the problem is that it's page.tpl.php has a div of id=menu.

It seems that somewhere in the page handling variables are created with the names matching these divs before the dhtml processing occurs and IE doesn't like the attempt at reassigning other values to them. Possibly these variables are references to other parts of the dom?

Unfortunately I don't really know anything about javascript or how the pages are built yet so I'm not up to providing a patch yet but I suspect the answer might be the dhtml module using more unique names for these variables.

Comment #4

cburschka

they

commented 2 November 2008 at 00:41

Ugh, IE really sucks. Have you tried renaming the variables to test this hypothesis?

Comment #5

cburschka

they

commented 5 November 2008 at 16:41

Status:

Active

» Postponed (maintainer needs more info)

Comment #6

cburschka

they

commented 5 November 2008 at 16:55

Status:

Postponed (maintainer needs more info)

» Needs review

Status	File	Size
new	dhtml_menu_6_2_ie_sucks-311886-6.patch	2.87 KB

I'm willing to change the variable names, but this has to be the last time. If any other themes are a problem, you'll just have to choose between them or IE. I can't keep changing locally declared variable names every time some browser decides it wants to reserve them.