javascript failed in Internet Explorer

[shuoshuo+trac@…]

1. Background
   • ​http://msdn.microsoft.com/library/default.asp?url=/workshop/author/dhtml/reference/methods/getattribute.asp
     • When retrieving the CLASS attribute using this method, set the sAttrName to be "className", which is the corresponding Dynamic HTML (DHTML) property.

2. Fix
   • ctxtnavadd.js

                         if(li_node.getAttribute('class') == 'last'
                             || li_node.getAttribute('className') == 'last') {
                             return(li_node);
                         }
     

[Dave Gynn]

I can verify this fix solves the problem in IE. You also need to make a similar change to the code that resets the 'last' item's class.

     new_li.setAttribute('class','last');
     old_li.removeAttribute('class');
+    new_li.setAttribute('className','last');
+    old_li.removeAttribute('className');

[jason_field]

I think there is a neater way to do it,

In find_old_last() instead of the getAttribute call;

if( li_node.className == 'last' ) {

and in add_ctxtnav() instead of the setAttribute call;

new_li.className = 'last'; old_li.className = ;

These changes seem to work on both IE and Firefox for me, and don't require doing essentially the same thing twice :)

[Noah Kantrowitz]

I've totally rework the JavaScript in the 0.11 version using jQuery. I don't really intend to backport these changes (as I am currently very lacking of free time), but if someone wants to make a backport patch, I'll take a look.

[Kenneth Xu]

fixed IE problem.

[Kenneth Xu]

I fixed this IE problem before I come to this ticket. file attached attachment:ctxtnavadd.js​

Can somebody help to check into 0.10 branch?

[Ryan J Ollos]

Plugin is deprecated.