Little improvement

[axbernardi]

Hi, to allow the possibility to use a variable in the text of the menu I modified row 38 in:

        textMenu = (itemOptions.text) ? itemOptions.text : me;
        var menuItem = $('<li><a href="#">' + textMenu + '</a></li>');

this allows to pass the parameter text with a possible variable.
Thanks for this very useful plugin

[arnklint]

Awesome! Looks clean to me. Have you tested it in the different browsers?

[axbernardi]

IE9, FF, and Chrome are definitively ok with it, didn’t try Opera but to be honest, I don’t see why it should have any problem... (though experience teaches that is better not to take anything for granted!)
Thanks for answering
Alessandro

From: Jonas Arnklint
Sent: Friday, September 14, 2012 7:06 PM
To: arnklint/jquery-contextMenu
Cc: axbernardi
Subject: Re: [jquery-contextMenu] Little improvement (#11)

Awesome! Looks clean to me. Have you tested it in the different browsers?

—
Reply to this email directly or view it on GitHub.

[arnklint]

Cool,
I´ll implement it and make sure you are among the contributors. Thanks axbernardi!

[axbernardi]

Thank YOU very much for the original work, it helped me so much in my applications (I’ve been using it for more than a year!).
Take care
Alessandro
(PS: if you want to attach a website to my name, use http://www.assetplan.co.uk )

From: Jonas Arnklint
Sent: Friday, September 14, 2012 7:21 PM
To: arnklint/jquery-contextMenu
Cc: axbernardi
Subject: Re: [jquery-contextMenu] Little improvement (#11)

Cool,
I´ll implement it and make sure you are among the contributors. Thanks axbernardi!

—
Reply to this email directly or view it on GitHub.