1

I'm trying to make one button that has two functions whether it has class in it or not when clicked. So I tried to make javascript code using if else statement. I don't see what the problem is, but the button is not working..

$(document).ready(function() {
  $(".btn-ctrl").click(function() {
    if ($(".btn-ctrl > span").hasClass("close")) {
      function popDown() {
        $("#banner").slideDown(400);
        $(".btn-ctrl > span").removeClass("close");
      }
    } else {
      function popUp() {
        $("#banner").slideUp(400);
        $(".btn-ctrl > span").addClass("close");
      }
    }
  })
})

.btn-ctrl is class name for my button and I'm changing the button's design by adding and removing class="close" in span tag. I can't figure out what is causing problem

3
  • 1
    Post your HTML. Commented Aug 11, 2021 at 13:21
  • 1
    Extract popUp and popDown functions from your event and call them in your if / else statement. If it doesn't work can you post your HTML ? Commented Aug 11, 2021 at 13:24
  • 1
    <button type="button" class="btn-ctrl"><span>POPUP</span> this is my html code for the button Commented Aug 11, 2021 at 13:26

1 Answer 1

2

You create functions inside the condition instead of calling them. Blame functions outside the condition block and then call them or remove them altogether. And before you write anything, read the syntax of the language.

$(document).ready(function() {
    $(".btn-ctrl").click(function() {
        console.log('click')
        if ($(".btn-ctrl > span").hasClass("close")) {
            $("#banner").slideDown(400);
            $(".btn-ctrl > span").removeClass("close");
        } else {
            $("#banner").slideUp(400);
            $(".btn-ctrl > span").addClass("close");
        }
    })
})

or

function popDown() {
    $("#banner").slideDown(400);
    $(".btn-ctrl > span").removeClass("close");
}
function popUp() {
    $("#banner").slideUp(400);
    $(".btn-ctrl > span").addClass("close");
}

$(document).ready(function() {
    $(".btn-ctrl").click(function() {
        console.log('click')
        if ($(".btn-ctrl > span").hasClass("close")) {
            popDown();
        } else {
            popUp();
        }
    })
})
Sign up to request clarification or add additional context in comments.

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.