feat: ctrl-s now saves on settingsv1/v2 forms

... because nobody ever uses ctrl-s on a browser window.
NodeBB · Sep 18, 2020 · 8093142 · 8093142
1 parent dc0a4a4
commit 8093142
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/public/src/admin/settings.js b/public/src/admin/settings.js
@@ -1,7 +1,7 @@
 'use strict';
 
 
-define('admin/settings', ['uploader'], function (uploader) {
+define('admin/settings', ['uploader', 'mousetrap'], function (uploader, mousetrap) {
 	var Settings = {};
 
 	Settings.init = function () {
@@ -90,6 +90,11 @@ define('admin/settings', ['uploader'], function (uploader) {
 			});
 		});
 
+		mousetrap.bind('ctrl+s', function (ev) {
+			saveBtn.click();
+			ev.preventDefault();
+		});
+
 		handleUploads();
 		setupTagsInput();
 

diff --git a/public/src/modules/settings.js b/public/src/modules/settings.js
@@ -509,6 +509,16 @@ define('settings', function () {
 					app.flags._unsaved = true;
 				});
 
+				var saveEl = document.getElementById('save');
+				if (saveEl) {
+					require(['mousetrap'], function (mousetrap) {
+						mousetrap.bind('ctrl+s', function (ev) {
+							saveEl.click();
+							ev.preventDefault();
+						});
+					});
+				}
+
 				callback(null, values);
 			});
 		},