Jquery how to run a function only if condition is true

Question

I have a function that check if CSS value exists, the problem is that I need the function to work only when the CSS class exists, currently the function running all the time because I'm using else condition (that need to re do the if condition).

 //use for @media only screen and (max-width: 767px) detection    
 $(window).resize(function(){
    if ($('#mobile-view').css('visibility') === 'hidden') {
        $("#product-gallery").insertAfter("#product-info");
    }
     else { 
         $("#product-info").insertAfter("#product-gallery");
    }
 });

Ja͢ck · Accepted Answer · 2013-04-23 09:13:42Z

3

You could use the :hidden pseudo selector:

if ($('#mobile-view').is(':hidden')) {
    $("#product-gallery").insertAfter("#product-info");
} else {
    $("#product-info").insertAfter("#product-gallery");
}

Comments

chandresh_cool · Accepted Answer · 2013-04-23 09:13:28Z

1

Change it to

if ($('#mobile-view').is(":visible")) {

    $("#product-info").insertAfter("#product-gallery");
} else {
     $("#product-gallery").insertAfter("#product-info");
 }

answered Apr 23, 2013 at 9:13

chandresh_cool

11.9k3 gold badges32 silver badges46 bronze badges

7 Comments

chandresh_cool Over a year ago

isn't it right if visible insert after product gallery or else insert after product-info?

Samuel Caillerie Over a year ago

it seems that the OP has posted the inverse condition : if #mobile-view is invisible then we insert the product gallery...

chandresh_cool Over a year ago

he say this if ($('#mobile-view').css('visibility') === 'hidden') { $("#product-gallery").insertAfter("#product-info"); } that means if hidden then insert after product info ? Correct?

Samuel Caillerie Over a year ago

Yes correct, but your logic is if visible, then insert after product info... :)

chandresh_cool Over a year ago

No :) if Visible do this $("#product-info").insertAfter("#product-gallery"); that is insert after product gallery.

|

DevelopmentIsMyPassion · Accepted Answer · 2013-04-23 09:14:02Z

0

you can use hasClass

if ($('#mobile-view').hasClass(className)) {
 // put your logic inside this.
}

answered Apr 23, 2013 at 9:14

DevelopmentIsMyPassion

3,6014 gold badges36 silver badges61 bronze badges

Comments

HasanAboShally · Accepted Answer · 2013-04-23 09:14:23Z

0

Try using $("#mobile-view").is(":visible") == true

answered Apr 23, 2013 at 9:14

HasanAboShally

18.8k9 gold badges32 silver badges35 bronze badges

Collectives™ on Stack Overflow

Jquery how to run a function only if condition is true

4 Answers 4

Comments

7 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

7 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related