Ocena kodu przewijania strony.

0

Witam. Postanowiłem napisać prosty skrypt który wyświetla i przwija strone do góry. Napisałem do w formie obiektu i proszę o opinie. Dopiero zaczynm przygode z js.

function $(x){
		return document.querySelector(x);
	}
	
	(function(){
	
	var Scroll = {
		element: $(".scroll"),
		display: function(){
			var $self = this.element;
			if (window.scrollY > 300){
				$self.classList.add("show");}
				    else { $self.classList.remove("show");}
				
			$self.onclick = function(){
				Scroll.top();
			}
		},
		top: function(){
			var $s = setInterval(function(){
				window.scrollBy(0, -10);
				if (window.scrollY <= 0){
					clearInterval($s);
				}
			}, 1);
		}
	}
	
	window.onscroll = function(){
		Scroll.display();
	}
	
})();
0

Nie wiem czy jak dodasz jquery do projektu (o ile dodasz) to czy nie będzie Ci się gryzło z dolarem biblioteki. AngularJS też chyba korzysta z jqLite ale on to chyba opakowuję, więc jest duża szansa, że nie ugryzie. Dla pewności zmieniłbym nazwę tego utila, na coś w rodzaju getSelector.
Tak wydaję się ok, chociaż wyjadacze JS-owi pewnie coś znajdą.

1 użytkowników online, w tym zalogowanych: 0, gości: 1