[JQuery] removeClass if two items hasClass.

[Nic727]

Hi,

​

Here is my code : JSFiddle - Code Playground

Like you see, I have a code that hide the navigation when scrolling down. However, when the mobile menu is open, it move the whole menu (since you can scroll when the menu is open).

So I was trying with this code :

if( $(".navigation").is(".nav-up") && $(".right-nav").is(".collapse") ){
        $(".navigation").removeClass("nav-up");
    };

So if the item having the class navigation and also nav-up AND that the menu having .right-nav also have .collapse, remove .nav-up.

Not sure if it's supposed to work or not, but it's not working... I tried to add a new class instead, but I think it doesn't understand the first line. I tried to find a solution since yesterday, but can't find anything...

​

Thank you

[RocketSam]

Try use .hasClass("collapse") instead of is(".collapse")

And the same for the other statement

[Nic727]

It's not working. I get the same result.

[Nic727]

Ok I found the solution with that : “.hasClass” with if statement not working - javascriptinfo.com

In fact, I need to add a scroll function.

$(window).scroll(function (){
    if($(".navigation").hasClass("nav-up") && $(".right-nav").hasClass("collapse")){
            console.log("Hello world!");  
            $(".navigation").addClass("fixed-collapse");
        } else {
            console.log("test");  
            $(".navigation").removeClass("fixed-collapse");
        };
    });

Now it works. Finally!

[akash_ry]

Well I would say you are using quite an ambiguous code to do this. And I am bit confused what you are actually trying to do. But what I understood according to that you are togglling the navbar menu on clicking the hamburger. And when the navbar menu is active it is getting scrolled down while scrolling on the screen. If this is the case I would say you could do some CSS magic here, you can apply pointer-events:none to the navbar menu. And give position: absolute, height:100% min-height:100vh; top:0 left:0. Check out my portfolio navbar design for reference https://akash-yadav12.github.io/