Redundant/unnecessary code overriding CSS properties on background image

[jtsimoes]

Hi!

Since in your .css you told us to add this:

universal-parallax/dist/universal-parallax.css

Lines 11 to 16 in 257276d

	.parallax__container .parallax {
	/* can be put in a seperate class for better control */
	background-position: center;
	background-repeat: no-repeat;
	background-size: cover;
	/* --------------------------- */

Why then you have an if on your .js doing the same thing?

universal-parallax/dist/universal-parallax.js

Lines 114 to 118 in 257276d

	if (parallax[i].classList.length === 1 && parallax[i].classList[0] === 'parallax') {
	parallax[i].style.backgroundRepeat = 'no-repeat';
	parallax[i].style.backgroundPosition = 'center';
	parallax[i].style.backgroundSize = 'cover';
	}

If no other class than .parallax is specified, adding this CSS it's kinda unnecessary code because the .parallax class already have it, no?

---

Also, this piece of code takes away the possibility of having repeated backgrounds with other size than cover and with a custom position (different then center). For instance, it's useful when you have small size GIFs or patterns and you need to repeat the background to fill the parallax:

Background image:
https://i.imgur.com/o4hJvT7.gif

Code:

<section style="min-height: 100vh;">
	(...)
	<div class="parallax" data-parallax-image="/img/403.gif" style="background-repeat: repeat; background-size: 500px;">
	</div>
</section>

Parallax before removing the if (if's code replacing the inline style I put on div):

Parallax after removing the if (same code it works now):

(The parallax effect was working on both, trust me)

[marrio-h]

Thank you @jtsimoes I will look into it (an all other comments) soon. If you want, feel free to make a pull request😊

All of your feedback is invaluable