Click button run function and addClass

Question

I'm trying to add a class to a bootstrap modal when a button is clicked. Nothing fancy except that there is only one modal which is populated with JSON content. To make my script working properly I need top add some identifier to the modal so the script knows which modal is open.

The modal looks like this:

<div data-show="false" data-width="800" tabindex="-1" aria-hidden="false" role="dialog" class="modal fade in" id="shop-modal">
    <div class="modal-dialog modal-lg">
      ... content from JSON ...
      </div> 
    </div>

The modal is opened like this:

<a data-vid="123456" data-handle="link-to-product.html" class="trigger-cart" href=""></a>

 $('.trigger-cart').click(function(e){
    e.preventDefault();
    var $this = $(this);
    var url = $this.data('handle') + '/?format=json';
    var vid = $this.data('vid');
    quick_shop(url, vid);
  });

The function which is triggered on button click is this:

function quick_shop(url, vid){

  $('#shop-modal').addClass(vid);

$.getJSON(url, function (data){
  $('#shop-modal').modal('show');
   // do stuff....

What i try to do is to add vid as a class to the opened modal. As I tried with this $('#shop-modal').addClass(vid); I also tried stuff like this:

$('#shop-modal').modal('show').addClass(vid);

This doesn't work. Does anyone know how to do that?

@JoshCrozier: Ok thx but vid is something like 123456. Why doesn't addClass(vid) works then? This works though, addClass(''+vid+'') — Meules
– Meules, Commented Sep 1, 2014 at 16:52
You're right. I read "I try to do is to add vid as a class" literally. It's worth noting that CSS classes starting with numbers are invalid - stackoverflow.com/questions/448981/… — Josh Crozier
– Josh Crozier, Commented Sep 1, 2014 at 17:13

Community · Accepted Answer · 2017-05-23 12:05:47Z

1

Based on your comment:

Ok thx but vid is something like 123456. Why doesn't addClass(vid) works then? This works though, addClass(''+vid+'')

It sounds like you are passing a number to addClass but addClass requires a string. That is why addClass(''+123456+'') works but not addClass(123456), the use of empty strings on either side of the number type coerce it into being a string.

If instead you did addClass((vid).toString()) it would also work. Although, I have to also mention that using just a number as a class name is not valid.

edited May 23, 2017 at 12:05

CommunityBot

11 silver badge

answered Sep 1, 2014 at 17:11

DerekK

613 bronze badges

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

Collectives™ on Stack Overflow

Click button run function and addClass

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related