Disabled menu items can be clicked

[abodnar]

After upgrading to 2010.2 825 I'm seeing that disabled menu items appear disabled but the user is able to click on them.  Is anyone else seeing this behavior?

[Georgi Krustev]

Hello abodnar,

Thank you for pointing this issue to us.

I confirm it as a bug. Fix will be included in the next official release of the Telerik Components for ASP.NET MVC. In order to fix it locally you need to change update click method in telerik.menu.js file:

[in telerik.menu.js]:

click: function (e, element) {
    e.stopPropagation();

    var $li = $(element);

    if ($li.hasClass('t-state-disabled')) {
        e.preventDefault();
        return;
    }

    $t.trigger(this.element, 'select', { item: $li[0] });

    if (!$li.parent().hasClass('t-menu') || !this.openOnClick)
        return;

    e.preventDefault();

    this.clicked = true;

    this.triggerEvent('open', $li);

    this.open($li);
},

I have updated your Telerik points.

All the best,
Georgi Krustev
the Telerik team

[abodnar]

Hi Georgi,

Thank you for your reply.  I'm still seeing some odd behavior with the menu items which is preventing your proposed fix from working correctly.  I've used Firebug to confirm that the disabled menu item does have the class 't-state-disabled', but when I click on the disabled menu item, the class for the referenced element in the click() function is 't-item t-state-default' Any ideas?

[Georgi Krustev]

Hello abodnar,

There were several changes in the telerik.menu.js file. In order to avoid any misunderstandings I have attached the telerik.menu.js file with the fix.

Regards,
Georgi Krustev
the Telerik team

[a c]

You can also modify the source code NavigationHtmlBuilderBase.cs:
        public IHtmlNode LinkTag(TItem item, Action<IHtmlNode> configure)
        {
            var url = Component.GetItemUrl(item) ;
           ....
Changed to:
        public IHtmlNode LinkTag(TItem item, Action<IHtmlNode> configure)
        {
            var url = item.Enabled ? Component.GetItemUrl(item) : "#";
           ....

[Carson]

@a c
Thanks! That saved me a ton of time.

[Mike]

Can i the minified version?  Or has this been release yet?

[Atanas Korchev]

Hi Mike,

 The latest official release should include the fix.

Greetings,
Atanas Korchev
the Telerik team