Toggle one element and add class to another element jquery

Question

I have left-side menu for mobile users. It should open on click on img#mobile-menu

I wrote such code

(function() {
    $("#img#mobile-menu").removeClass("active");
    $("img#mobile-menu").toggleClass("active");
    if ($("img#mobile-menu").hasClass("active")) {
        $("#mobile-menu-sidebar").addClass("active");
    } else {
        $("#mobile-menu-sidebar").removeClass("active");
    }
}());

Problem is that when I open / refresh page both elements #mobile-menu and #mobile-menu-sidebar already has active class, but they shouldn't have it till I click img#mobile-menu

do it on click event of the menu

guradio
– guradio

2017-01-17 09:08:20 +00:00
Commented Jan 17, 2017 at 9:08 — guradio
– guradio, Commented Jan 17, 2017 at 9:08

guradio · Accepted Answer · 2017-01-17 09:12:06Z

1

$("#mobile-menu").toggleClass("active");
$('#mobile-menu').click(function() {
  $("#mobile-menu").toggleClass("active");
  if ($("#mobile-menu").hasClass("active")) {
    $("#mobile-menu-sidebar").addClass("active");
    $('span').text("mobile-menu-sideba has active ")
  } else {
    $("#mobile-menu-sidebar").removeClass("active");
    $('span').text("mobile-menu-sideba has remove active ")
  }
})

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<button id='mobile-menu'>mobile-menu</button>
<span></span>

Do it no click event

answered Jan 17, 2017 at 9:12

guradio

15.6k4 gold badges39 silver badges65 bronze badges

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

2 Comments

angelzzz Over a year ago

tried your variant. but when I refresh page img#mobile-menu already has class active

guradio Over a year ago

remove that line before click event

Ahmad · Accepted Answer · 2017-01-17 09:10:42Z

0

have the first line of your function disable/remove-active from both elements:

$("#img#mobile-menu,#mobile-menu-sidebar").removeClass("active");

answered Jan 17, 2017 at 9:10

Ahmad

12.8k7 gold badges53 silver badges98 bronze badges

1 Comment

angelzzz Over a year ago

thank you. didn't try your example as used another one

Collectives™ on Stack Overflow

Toggle one element and add class to another element jquery

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related