jQuery being annoying...

[Roon]

So I'm trying to code a login/logout popover like so...

    $(document).ready(function(){
        $('.content-login-popover').hide();
    });
    $(document).ready(function(){
        $('#login').click(function (){
            $('.content-login-popover').show();
        }));                                                                     <--------- LINE 7.
    };
$(document).ready(function(){
    $('.content-login-popover').hide();
});
$(document).ready(function(){
    $('#login').click(function(){
        $('.content-login-popover').show();
    }));
});

However, when I do this, I'm getting an error saying...

"15:11:17:845 - SyntaxError: missing ; before statement - main-scripts.js:7"

Thanks in advance

 

[NSA]

Missing bracket on line nine.

 

[Sysode]

Like

You must be registered for see links

said, why do you have three brackets?

Anyway, try this;

 $(document).ready(function(){
        $('#login').click(function (){
            $('.content-login-popover').show();
        }));                                                                    <--------- LINE 7.
    };

Change to:

 $(document).ready(function(){
        $('#login').click(function (){
            $('.content-login-popover').show();
        });                                                                    <--------- LINE 7.
    });

 

[Markshall]

Not quite sure why you did $(document).ready, did one line of code and then ended it and restarted it 4 times. This will suffice.

You also had two unneeded brackets.

$(document).ready(function(){
    $('.content-login-popover').hide();
 
    $('#login').click(function(){
        $('.content-login-popover').show();
    });
 
    $('.content-login-popover').hide();
 
    $('#login').click(function(){
        $('.content-login-popover').show();
    });
});

 

[Roon]

I didn't realize that there was another way to do jQuery, I thought you had to keep saying "if document is ready do this..." for each thing.
What is the more 'professional' way?

 

[Markshall]

I didn't realize that there was another way to do jQuery, I thought you had to keep saying "if document is ready do this..." for each thing.
What is the more 'professional' way?

Not to blow my own trumpet, but the more 'professional' way is the way I did it, where you call jQuery once on document.ready and then start using it properly to do tricks and shit, yo.