Switch to ESM

[erjonify]

closes #185

[jimmywarting]

StreamSaver.js/StreamSaver.js

Lines 5 to 12 in 5b372f7

	;((name, definition) => {
	typeof module !== 'undefined'
	? module.exports = definition()
	: typeof define === 'function' && typeof define.amd === 'object'
	? define(definition)
	: this[name] = definition()
	})('streamSaver', () => {
	'use strict'

• Needs to be removed (and the end of it as well)

• The script tag must be changed to have the type="module"

We can remove all of this:

<script src="https://cdn.jsdelivr.net/npm/web-streams-polyfill@2.0.2/dist/ponyfill.min.js"></script>
<script src="https://cdn.jsdelivr.net/npm/streamsaver@2.0.3/StreamSaver.min.js"></script>
<script>
  import streamSaver from 'streamsaver'
  const streamSaver = require('streamsaver')
  const streamSaver = window.streamSaver
</script>

and replace it with:

<script type="importmap">
  {
    "imports": {
      "streamsaver/StreamSaver.js": "https://cdn.jsdelivr.net/npm/streamsaver@3/StreamSaver.min.js"
    }
  }
</script>
<script type="module">
  // either one works fine, the later one requires importmaps or a web bundler. 
  import streamSaver from 'https://cdn.jsdelivr.net/npm/streamsaver@3/StreamSaver.min.js'
  import streamSaver from 'streamsaver/StreamSaver.min.js'

  const uInt8 = new TextEncoder().encode('StreamSaver is awesome')
  ...
</script>

• Every green browser now days have good support for web streams, so i think we can remove the hole shebang where we mention any use of custom stream polyfill

// StreamSaver can detect and use the Ponyfill that is loaded from the cdn.
streamSaver.WritableStream = streamSaver.WritableStream
streamSaver.TransformStream = streamSaver.TransformStream

• also the code examples in https://github.com/jimmywarting/StreamSaver.js/tree/master/examples needs to be changed to also have type="module" in their script tags

[jimmywarting]

☝️

[erjonify]

• type="module"

@jimmywarting , i did some small changes, feel free to complete the rest :)

[PekingSpades]

If you're using next.js, etc., make sure you dynamically bring in streamSaver.js. the way to do this is to import('. /StreamSaver').then

[bytes-max]

Will it ever be merged? :)