From a19e6f26e151b159f20fd7ed86938e0baedf7f20 Mon Sep 17 00:00:00 2001 From: Frederik Date: Thu, 13 Jun 2019 14:29:14 +0200 Subject: [PATCH] Use CTRL+Enter on Linux (#121) Currently, there's only "Windows and "not Windows", which makes all non-Windows platforms behave like Mac (i.e., they have a meta key.). That's unfortunately not very true. Instead, I'd suggest use the apple meta key on platforms that are known to be Mac and support CTRL+Enter on all other platforms (Linux, Unix, etc.) --- client/js/dpaste.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/client/js/dpaste.js b/client/js/dpaste.js index c75d32c..560e980 100644 --- a/client/js/dpaste.js +++ b/client/js/dpaste.js @@ -3,8 +3,8 @@ // ----------------------------------------------------------------------------- // Add data-platform to the body tag to show platform related shortcuts // ----------------------------------------------------------------------------- -const isWindows = navigator.appVersion.indexOf('Win') !== -1; -document.body.dataset.platform = isWindows ? 'win' : 'mac'; +const isMac = navigator.platform.indexOf('Mac') !== -1; +document.body.dataset.platform = isMac ? 'mac' : 'win'; // ----------------------------------------------------------------------------- // Autofocus the content field on the homepage @@ -18,7 +18,7 @@ if (af !== null) { // Cmd+Enter or Ctrl+Enter submits the form // ----------------------------------------------------------------------------- document.body.onkeydown = function(e) { - const metaKey = isWindows ? e.ctrlKey : e.metaKey; + const metaKey = isMac ? e.metaKey : e.ctrlKey; const form = document.querySelector('.snippet-form'); if (form && e.keyCode === 13 && metaKey) { @@ -141,4 +141,4 @@ if (editSnippetLink && editSnippetForm) { editSnippetForm.getBoundingClientRect().y ); }; -} \ No newline at end of file +}