'Let' itself has a block level scope, function scope is not required

debug/simplemde.js

@@ -16925,15 +16925,17 @@ var SimpleMDE = function (_Action) {
 			var self = this;
 			var keyMaps = {};
 
-			for (var key in options.shortcuts) {
+			var _loop = function _loop(key) {
 				// null stands for "do not bind this command"
 				if (options.shortcuts[key] !== null && _metadata.bindings[key] !== null) {
-					(function (key) {
-						keyMaps[_utils2.default.fixShortcut(options.shortcuts[key])] = function () {
-							return _metadata.bindings[key](self);
-						};
-					})(key);
+					keyMaps[_utils2.default.fixShortcut(options.shortcuts[key])] = function () {
+						return _metadata.bindings[key](self);
+					};
 				}
+			};
+
+			for (var key in options.shortcuts) {
+				_loop(key);
 			}
 
 			keyMaps["Enter"] = "newlineAndIndentContinueMarkdownList";

src/js/simplemde.js

@@ -179,9 +179,7 @@ class SimpleMDE extends Action {
 		for(let key in options.shortcuts) {
 			// null stands for "do not bind this command"
 			if(options.shortcuts[key] !== null && bindings[key] !== null) {
-				(function(key) {
-					keyMaps[utils.fixShortcut(options.shortcuts[key])] = () => bindings[key](self);
-				})(key);
+				keyMaps[utils.fixShortcut(options.shortcuts[key])] = () => bindings[key](self);
 			}
 		}