Memory leak in Dialog.open()

[sunnywalker]

accessible-modal-dialog/Dialog.js

Line 26 in b5746cb

this.dialogEl.addEventListener('keydown', function(e) {

You've got a memory leak in Dialog.open(). Every time a dialog is opened, a new keypress event handler is added to the dialog and a new click handler to the overlay. If a dialog is opened 100 times, that is 200 event handlers for the browser to process. For a navigation menu, this is unlikely, but for other modal dialog uses, it could potentially become problematic.