Fix compatibility with older browsers in zxing-js

[bilal-elchami]

This PR fixes the described bug in this issue: #188

@all-contributors please add @bilal-elchami for this feature.

[mebjas]

Thanks for the change.

Please resync once more - I had a commit which should remove the minified file generation from npm build.

Alternatively please revert the minified/html5-qrcode.min.js change in this PR.

I'll generate the new one and publish to npm and Github as part of release process.

[mebjas]

Thanks for the change -- merging.

[JL102]

@mebjas This change breaks html5-qrcode in modern browsers. Tested on Vivaldi 5.7.2921.63 (using V8 11.0.226.19) and Firefox 111.0. Built from source on Ubuntu and imported into a Svelte project to test.

[bilal-elchami]

@mebjas This change breaks html5-qrcode in modern browsers. Tested on Vivaldi 5.7.2921.63 (using V8 11.0.226.19) and Firefox 111.0. Built from source on Ubuntu and imported into a Svelte project to test.

@JL102 Sorry to hear this. I will try to find a fix by the end of the day.

[mebjas]

I am reverting this by another PR for now (#738)

@bilal-elchami if you have a non breaking change later - please send another PR.

[mebjas]

@JL102 thanks for sharing the issue.

[JL102]

@mebjas np. Try this instead:

const g = (typeof globalThis !== 'undefined') ? globalThis :
    (typeof global !== 'undefined') ? global :
    (typeof self !== 'undefined') ? self :
    (typeof window !== 'undefined') ? window : {};
var __awaiter = g.__awaiter || function ( .....

The typeof operator allows non declared variables.

[khata-anshul]

Can we get these changes merged please - #700 (comment)

v2.3.8 is still throwing globalThis is not defined error

[axylophon]

This would be really nice as we are currently blocked with anything below Android 10...

I have added the changes suggested from @JL102 and created the PR #794