Access window and documents in function scope

[meteorlxy]

Bug description

In v0.4.0, we only try to access window / document in the function scope, i.e. all related code is wraped in the mediumZoom function:

medium-zoom/src/medium-zoom.js

Lines 29 to 39 in 8617c95

	const mediumZoom = (
	selector,
	{
	margin = 0,
	background = '#fff',
	scrollOffset = 48,
	metaClick = true,
	container,
	template
	} = {}
	) => {

In v1.x, we access window / document outside the mediumZoom function:

medium-zoom/src/medium-zoom.js

Lines 104 to 111 in e1b85df

	const Promise =
	window.Promise ||
	function Promise(fn) {
	function noop() {}
	fn(noop, noop)
	}
	const mediumZoom = (selector, options = {}) => {

I'm using medium-zoom in a server-rendered project, but window / document can only be accessed in client. The differences cause errors when I try to migrate to v1.x from v0.4.0.

If it won't be fixed, I think it is one of the breaking changes that should be listed.

[francoischalifour]

Thanks for filing an issue.

There's a simple fix to this without actual tradeoff so we might actually want to add support for SSR here.

Would you mind opening a PR?

[meteorlxy]

With pleasure 😄