Opened 2 years ago
Last modified 1 year ago
1. Background
2. Fix
if(li_node.getAttribute('class') == 'last' || li_node.getAttribute('className') == 'last') { return(li_node); }
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');
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 :)
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.
fixed IE problem.
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?
