Milonic JavaScript Menu

Ironically ianpreston exhibits the faults he perceives in JonkieXL and me; but his patronising complaint does nonetheless raise a pertinent point, despite aparently being totally irrelevant.

Andy is welcome to ignore all these bug reports if he wants (he's already disregarded several of mine), but ...

Having a single in your URL will cause mmenu.js to error.

Simple to reproduce, just take the menu_array.js from the standard menu.zip and change:
,"CNN","http://www.cnn.com",,,0

to:
,"CNN","http://www.cnn.com?hello='world'",,,0

I can understand how that would compensate for the bug in mmenu.js, but I would rather see this fixed in mmenu.js than use this work around. While what you've suggested may solve the problem, it seems like a kludge to me and just doesn't seem "right" long term.

Should I assume that you don't intend ...

I believe the string literal in my example is syntactically correct as JavaScript.

Using the escape sequence as suggested to construct the menu_array would give the same results because it would represent the same string. As it happens, I did post both cases before making my original post.

FYI ...

When a single quote is inserted into the Status for a menu item, an error occurs immediately when the page is loaded. This breaks for me using IE6, but doesn't break with Mozilla 1.0.

This is easy to reproduce, get the standard menu.zip and in menu_array.js change

,"Home","http://www.dynamicdrive ...

My two requirements are as follows:

a) open menus onclick
b) menu openers invoke my JavaScript URLs when clicked on

Each of these can be fulfilled, but not in combination. The option of opening menus onclick breaks the ability to invoke an alternate URL.

This has been a problem for some time and ...

Got this working now. Seems to be a documentation problem. I suggest that an example which uses menu openers is added to the Function Activation Example.

I have this working fine for most menu items, but with menu openers the menu isn't opened and the function isn't called either.

Have moved the target as suggested, and that helps now thanks. Without the target there at all, the URL doesn't open either. Perhaps it would help to update the FAQ to make it clear that setting the target is necessary for this to work. The FAQ makes no mention of this IIRC.

Anyway, this still ...

Set an alternative URL on a menu opener, and the menu won't open any more. This is what alternative URLS are for isn't it?

Using the demo code site in http://www.milonic.com/menu/menu.zip, I changed the "News & Sites" menu opener thus:

,"News sites ","show-menu=news target=_blank","http://www ...

Unfortunately 3.4.10 does not solve this problem either.

Will look forward to further updates!

I assume you mean version 3.4.09 (and that 3.3.09 is a typo).

I have downloaded 3.4.09 and found that it is still a problem. I've now found that the menu link is being fired up but that the menu isn't being opened instead! It ought to be doing both really.

I hope your subsequent attempt at fixing ...

You have essentially rephrased my original bug report, and come to the same conclusion I did.

I would like for only one axis of the positioning to be changed. I don't see how this is currently possible, without me having to calculate (for each sub menu) where it would have been placed in the other ...

Just because you think it should work one way, doesn't mean everyone wants it that way. I certainly don't, and would prefer the choice.

I still think that the option I have chosen should be honoured, since it is claimed this menu is "highly flexible". I find this to be an aspect which makes it ...

The FAQ claims that a DHTML menu element can act as a menu link as well as a menu opener.

This does not work when you have also chosen the option to "Open the Menus OnClick".

With the option to "Open the Menus OnClick", these elements only act as menu openers and the menu link is disregarded.

It doesn't seem possible to change the "Screen Position" of a sub menu in only one axis at a time.

Setting "Screen Position" to "left", seems to be equivalent to "top;left". I would not have expected that setting it to be "left" to change the vertical positioning as well! The same is also true of ...

I'm trying to use the option to "Open the Menus OnClick".

This seems to work fine for a top level menu, but the option seems to be totally ignored by sub menus.

I'm confident that I've selected the right option, since making my sub menu into the main menu causes the option to be recognised.