[P.Conrad]

Hi,

Basically I'm just starting my adventure with Jquery and need a bit of help from you guys.

How to shorten that code to one function cos I'm a profound believer in DRY (Don't repeat yourself).

$(".gotAccount").click(function(event){
		event.preventDefault();

		$(".register").fadeOut( function() {
			$(".login").fadeIn('slow')
			});
	});

	$(".dontHaveAccount").click(function(event){
		event.preventDefault();

		$(".login").fadeOut( function() {
			$(".register").fadeIn('slow')
			});
	});

It simply switches two dives. In div login we have login form and a link .dontHaveAccount, and in div register we have registration form with link .gotAccount.

Thanks guys!

[modi123_1]

Why not have a function that takes in a target (your .register or .login), takes in a fadein target, and a fade in speed.. then just call that function in each event?

[Dormilich]

another simplification:

$(".gotAccount, .dontHaveAccount").on("click", function(event) {
    // ...
});

and then something with toggle().

This post has been edited by Dormilich: 22 May 2013 - 07:17 AM

[P.Conrad]

I tried something like that Dormilich, but problem is I want to first fadeOut one div and then fadeIn the other. So if I use toggle it will work only one way - the other would be that second div fadeIn and only after the other fadeOut (so for a while I would have two divs visable - which is not necceserly what I'm after) . The solution would be, if I have your code

$(".gotAccount, .dontHaveAccount").on("click", function(event) {

to asses which precise link was clicked and then use if statement. But can I acquire that in jquery ?

[Dormilich]

P.Conrad, on 24 May 2013 - 01:27 PM, said:

to asses which precise link was clicked and then use if statement.

in an event handler this refers to the current element (e.g. the element clicked).