Script Error at Internet Explorer 11 with Version 3.x

[liquid-a]

I have a script Error when i usw the mmenu.js with version 3.0.1 or 3.0.2

The console shows:
SCRIPT1002: Syntaxfehler
Datei: mmenu.js, Zeile: 1, Spalte: 2278

The menu is not functional with IE 11. No Problems with Firefox or Chrome.

When i use Verison 2.1.1 everything ist O.K.

[fritzmg]

mmenu in general is not compatible with IE 11. There is a PR for automatically including Polyfills though: #40
I'll release it as 3.1.0 today.

[Defcon0]

The problem here is with

document.addEventListener(
        "DOMContentLoaded", () => {
            new Mmenu('#<?= $this->elementId ?>', <?= json_encode($this->options) ?>, <?= json_encode($this->configuration) ?>);
        },
    );

being the js-template for loading the menu. Why not simply use function() {} instead of () => {} not working in IE11?

[Defcon0]

Also the trailing comma leads to a syntax error in IE11. So the more compatible version would be:

document.addEventListener(
        "DOMContentLoaded", function() {
            new Mmenu('#<?= $this->elementId ?>', <?= json_encode($this->options) ?>, <?= json_encode($this->configuration) ?>);
        }
    );

[fritzmg]

Yeah, I can optimize this. Also it was a mistake to call the template js_mmenu, as it is not a JavaScript Template that can be included in the page layout.

The code is directly pulled from the documentation of mmenu btw.

[operatorone]

Is it already working in IE11. i just implemented the module and activated the polyfill, but the navigation is jut thrown on the page and the trigger doesn't work.

[fritzmg]

@operatorone the problems mentioned by @Defcon0 have not been fixed yet.

[contaoacademy]

@fritzmg Do you know, when you can fix this?

[fritzmg]

@contaoacademy pls create a justom js_mmenu template and replace

contao_dk_mmenu/src/Resources/contao/templates/js_mmenu.html5

Lines 15 to 19 in 3f49fc0

	document.addEventListener(
	"DOMContentLoaded", () => {
	new Mmenu('#<?= $this->elementId ?>', <?= json_encode($this->options) ?>, <?= json_encode($this->configuration) ?>);
	},
	);

with:

    document.addEventListener(
        "DOMContentLoaded", function() {
            new Mmenu('#<?= $this->elementId ?>', <?= json_encode($this->options) ?>, <?= json_encode($this->configuration) ?>);
        }
    );

If this change works, then I'll change it immediately.

[contaoacademy]

I get this error

[contaoacademy]

After using the polyfill option the fix works.

[fritzmg]

@contaoacademy pls create a justom js_mmenu template and replace

contao_dk_mmenu/src/Resources/contao/templates/js_mmenu.html5

Lines 15 to 19 in 3f49fc0

	document.addEventListener(
	"DOMContentLoaded", () => {
	new Mmenu('#<?= $this->elementId ?>', <?= json_encode($this->options) ?>, <?= json_encode($this->configuration) ?>);
	},
	);

#53