donee

this is review message.

  1. PROPER EVENT BINDING: Consider using the preferred .on() method rather than .click(), .bind(), .hover(), .load(), .ready(), etc. For best performance and concise code use event delegation whenever possible: https://codepen.io/32bitkid/post/understanding-delegated-javascript-events
  1. INDENTATION: please make sure that your code it’s correctly indented, here in Envato we are committed to deliver the best product possible.
$('#divID').on('scroll',function(e){
   //your codes
});

Change your jQuery codes as above for all methods ( .click(), .bind(), .hover(), .load(), .ready(), etc )

what is this INDENTATION: please make sure that your code it’s correctly indented, here in Envato we are committed to deliver the best product possible.

http://jsbeautifier.org

?? fix all

where i need to change

This part you need to modify:

$(window).on('scroll', function() {
	if ($(this).scrollTop() > 500) {
	$('.scrollup').fadeIn();
	}
	else {
	$('.scrollup').fadeOut();
	}
});

need any code to change using .on @ki-themes

This part looks fine. If you have another JQ file, check that, too

okay thank you very much :slight_smile: