Jquery script not working with IE

Question

I written this script, to add JavaScript functionality to my online shop, the script works fine with Firefox and Chrome, but will not run on ie, and i am not sure why?

i am using jQuery(function( $ ){ instead of .ready due to script conflicts, i have tested the script using .ready and it still does not work with ie.

if anyone has an ideas they would be much appreciated.

jQuery(function( $ ){

setInterval(function(){ updatecart();  },8000);

   $('.addtobag').on('click', function(){
       event.preventDefault();
       var postdata = new Object();

       var action = $(this).closest('form').attr('action');

      $(':input',$(this).closest('form')).each(function(evt){

        var L = $(this).attr('name')
        postdata[L] = $(this).val();

      });


       $.post(action, postdata);

       generate('success'); //display banner
        updatecart();   //update cart

            });

    var postdata = new Object();

    postdata['basket'] = phpbasket;



     function updatecart() {

      $.post("/get_cart_details.php", postdata, function (data) {
                      var obj = $.parseJSON(data);
                           $('#qty_js').text(obj.items_qty);
                           $('#amt_js').text(obj.items_value);
                           });

     }



  function generate(type) {
    var n = noty({
        text: 'The item(s) have been added to your basket.',
        type: type,
      dismissQueue: true,
        layout: 'topCenter',
        theme: 'defaultTheme'
    });
    console.log('html: '+n.options.id);

         setTimeout(function() {
      $.noty.closeAll();
    }, 5000);
  }




});

Which version of jQuery, and which version of IE are you using? — Phil
– Phil, Commented Jan 30, 2013 at 22:15
You are missing a semi colon on this line "var L = $(this).attr('name')", but that's probably not it. Does the jQuery ready callback fire to even attach the event and is the element $('.addtobag') found? — matt snider
– matt snider, Commented Jan 30, 2013 at 22:20

j08691 · Accepted Answer · 2013-01-30 22:20:20Z

1

Get rid of the console.log() statement. IE < 9 chokes on it and 9 only works if the console is open.

answered Jan 30, 2013 at 22:20

j08691

208k33 gold badges269 silver badges281 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Musa · Accepted Answer · 2013-01-30 22:16:36Z

0

Try adding an event parameter in your click callback

$('.addtobag').on('click', function(event){

answered Jan 30, 2013 at 22:16

Musa

97.9k17 gold badges123 silver badges144 bronze badges

Comments

vaporbook · Accepted Answer · 2013-01-30 22:23:11Z

0

A couple things that might be tolerated in some browsers and not IE are that line 12 is missing a semicolon at the end, and the variable phpbasket is not defined (although if you defined it outside of the closure with 'var phpbasket' then you should be ok. If you debug it in IE 9 or higher, you should be able to see line numbers of errors in the console.

answered Jan 30, 2013 at 22:23

vaporbook

2751 silver badge7 bronze badges

Comments

Eric Sparks ii · Accepted Answer · 2013-12-05 16:17:51Z

0

Not a complete answer, but it may point you in the right direction. I was actually about to ask a similar question - I found how to get IE 11 to load the jquery, but it required the user to hit "f12" and then "run activex" because apparently IE considered my code potentially unsafe. But when the user tells the activex to run, it works fine.

I am trying to learn how to use jquery to make touchscreen-friendly dropdown menus, so my code was basic - no css, no doctype, just basic html and js. If anyone else could shed some light on this it would be great.

edited Dec 5, 2013 at 16:17

answered Dec 5, 2013 at 15:12

Eric Sparks ii

355 bronze badges

Collectives™ on Stack Overflow

Jquery script not working with IE

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related