fix: shouldResetPlayer options spread

[staff0rd]

Reset is currently broken, mostly due to the position of the playerVars spread. I also added a test to prove this.

Prior to this fix, any change to start was still resulting in a player reset.

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit c8beb15:

Sandbox	Source
Vanilla	Configuration

[ruisaraiva19]

@staff0rd This looks good. Let's wait for #278 to be merged first.

[tjallingt]

Looks great, thanks!

[ruisaraiva19]

@staff0rd can you sync your fork with the latest changes from this repo before I can merge this?

[github-actions]

🎉 This PR is included in version 7.13.1-canary.4 🎉

The release is available on:

• GitHub release
• npm package (@canary dist-tag)

Your semantic-release bot 📦🚀

[github-actions]

🎉 This PR is included in version 7.13.2-canary.1 🎉

The release is available on:

• GitHub release
• npm package (@canary dist-tag)

Your semantic-release bot 📦🚀

[github-actions]

🎉 This PR is included in version 7.14.0 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

[iyzana]

@staff0rd Do you still know why you added prevProps.videoId !== props.videoId to shouldResetPlayer?
From what I've observed, changing just the videoId works perfectly fine without resetting the player.
With this check the player unnecessarily leaves fullscreen when the video changes.