Needs work
Project:
tinynav.js
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Dec 2012 at 17:10 UTC
Updated:
6 Mar 2013 at 16:05 UTC
The code that tinynav puts in the page header is triggering a JS warning in IE8 and below. The problem code looks a bit like:
jQuery(document).ready(function($){
$('#nice-menu-1').addClass('tinyjs');
$('.tinyjs').tinyNav({
header: true,
});
});
I believe that it's the trailing comma at the end of header: true, that's causing the problem.
Comments
Comment #1
btopro commentedThat sounds possible; can you do me a favor and run it through http://www.jslint.com/ until it passes.
Comment #2
user1252762 commentedi have a problem with ie8 too. drop down doesn't create with ie8 and jquery1.7.
but if i update jquery to 1.8 via jquery_update drop down create.
there is only error i get with ie8 and drupal 1.7 and latest dev in ie8.
Unable to get value of the property 'slice': object is null or undefinedif remove this line of tinynav
error disappear so i think the error is related to tinynav.
i hope this clues would help.
Comment #3
btopro commentedsounds like you are saying if you remove the tinynav code that it works.
Comment #4
user1252762 commentedsorry for bad english.
i mean if i remove that piece of code, error disappear and tiny nav obviously doesn't work. my point was that error is probably related to tinynav.
Comment #5
btopro commentedsee comment 1. please run this through jslint.com and clean up whatever the issue is til it passes. It's probably some silly syntax thing if it's only causing issues in IE