Comments on: Proto.Menu: Prototype-based Context Menu Solution

By: shyamkkhadka

shyamkkhadka — Wed, 19 Aug 2009 10:10:55 +0000

This proto menu overrides browser default behaviour, means showing options like copy,paste,view source on right click.
How to avoid this problem?
Please suggest me

By: cinis

cinis — Tue, 16 Jun 2009 07:04:40 +0000

Hi,
the menu does not work with IE8. it works only in ‘Compatibility view’

By: kowadis

kowadis — Fri, 08 Aug 2008 15:51:43 +0000

Hi,
Thanks for a providing this.
Looks like I am almost a year late in noticing this script :)
Anyway,
I ran into a small issue using this in an IFRAME.
I am invoking menu when user clicks on links (I modified code a bit to use left mouse click on IE6+/FF). It works fine so far as page is loaded directly. When I load the same page in an IFRAME though, it fails to invoke the menu.
It seems the mouse click is not intercepted

Do anyone know how to fix this?
Thanks in advance.

By: Ronan

Ronan — Thu, 25 Oct 2007 10:18:33 +0000

Does anyone know how I can reference the ID of the element where the right-click event was called?

By: GG

GG — Tue, 28 Aug 2007 02:10:04 +0000

The script works with my firefox (v2.0.0.6) even when I turned the “disable script to replace context menu” options on.

It seems that is a bug of firefox/mozilla …
Bug 330756

By: Les

Les — Mon, 27 Aug 2007 16:10:45 +0000

kangax, I’ll be looking forward to release .6 :)
Also, I think it would be helpful if the callback had and argument with mouse position info (left and top).
Thx,
Les

By: kangax

kangax — Mon, 27 Aug 2007 15:22:29 +0000

@Mike
The cursor style is changed with CSS, so you’re free to choose any you want.
@Viktor
Can you tell which platform/browser you’re using? The browser menu should not interfere with the JS one (since default “contexmenu” event behaviour is being blocked)
@Martin
Yes, fading, indeed, could be annoying that’s why I made it completely optional. If you specify ‘fade: false’ option, menu will be appearing immediately.
@Les
Thanks, Les. This is exactly how I’m defining a class in a new version of a script (0.6). There are also few speed improvements coming, so stay tuned : )
@DWJ
Unfortunately, Opera does not allow to block browser’s context menu, nor does event prevention seems to work, therefore this ugly workaround.

By: DWJ

DWJ — Mon, 27 Aug 2007 08:45:25 +0000

Really nice implementation, the only con is that CTRL+left click stuff on Opera.

By: Les

Les — Sun, 26 Aug 2007 17:57:57 +0000

This is a very nice menu… and I’m planning to use it.

Just one very minor suggestion: Prototype 1.6 provides a simpler way to create classes, so you don’t have to use ‘Proto.Menu.prototype =’.

See an example below:

var Animal = Class.create({
initialize: function(name) {
this.name = name;
},
eat: function() {
return this.say(“Yum!”);
},
say: function(message) {
return this.name + “: ” + message;
}
});

By: Martin

Martin — Sun, 26 Aug 2007 15:16:28 +0000

Thanks for the script, that can be useful.

I would drop the fade in. It gets annoying for the user to wait (see navigating the Windows XP start menu). This is what OS X gets right. Show immediately and fade out (fast), Windows does it the other way around and it is not effective.

Note: Opera in OS X shows the menu on Command + click.

But what confuses me most is that it shows the menu at all in Firefox. I do NOT have the option “Allow scripts to disable or replace context menus” checked, so the script should not be able to replace the context menu!?!

By: Viktor

Viktor — Sun, 26 Aug 2007 11:51:33 +0000

Am I the only one that gets the regular browser context menu, with this js menu underneath the real one?

By: Mike

Mike — Sun, 26 Aug 2007 11:22:37 +0000

The text on that page no longer has the regular text cursor in Firefox 2.0 Is that because of the script?

By: kangax

kangax — Sun, 26 Aug 2007 05:47:42 +0000

Jordan,

You can’t modify container layout via current API. I just wanted to keep it as simple as possible :)
I’m thinking about adding option to specify className for each link – that will allow to create image based links (to give them more of a desktop look)

As far as Safari 2.0 – I unfortunately am not able to test it on that browser, and that’s the only browser I didn’t test (so I guess it was a little foolish to claim full A-Graded support)

I would definitely like to fix whatever it is that Safari 2.0 doesn’t like, if somebody could point to the right direction.

Cheers,
kangax

P.S. I’m glad people find this script useful, I’ll be definitely working on it

By: Jordan

Jordan — Sun, 26 Aug 2007 04:53:07 +0000

How can I customize it to have a “rounded corner” look? It doesn’t create additional DIVs for me to set background images on.

And not to be picky but isn’t Safari 2.0 an A-Grade browser? It’s the only non-beta version of Safari bundled on Macs.

By: emehrkay

emehrkay — Sat, 25 Aug 2007 19:32:43 +0000

Nice, but when things break a user’s typical UI behavior (right-click copy paste and such) It typically isnt a good thing. Hopefully devs who choose to use this will consider those things, like Google did with their right-click menus.

By: david

david — Sat, 25 Aug 2007 19:23:41 +0000

Very nice, very simple.

By: Alex Egg

Alex Egg — Sat, 25 Aug 2007 19:10:44 +0000

very high quality js. i like.