From 5f4ee049c673d6c0122d36f5572f632c23023c2b Mon Sep 17 00:00:00 2001
From: Mike Dalessio <mike@37signals.com>
Date: Thu, 28 Aug 2025 11:47:21 -0400
Subject: [PATCH] Fix up hotkeys

ref: https://fizzy.37signals.com/5986089/collections/2/cards/1452
---
 app/helpers/hotkeys_helper.rb       |  4 ++--
 test/helpers/hotkeys_helper_test.rb | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/app/helpers/hotkeys_helper.rb b/app/helpers/hotkeys_helper.rb
index 48f3cc6f7..062f6ca2d 100644
--- a/app/helpers/hotkeys_helper.rb
+++ b/app/helpers/hotkeys_helper.rb
@@ -8,7 +8,7 @@ module HotkeysHelper
         platform.mac? ? "return" : "enter"
       else
         key
-      end
-    end.join()
+      end.capitalize
+    end.join("+").gsub(/⌘\+/, "⌘")
   end
 end
diff --git a/test/helpers/hotkeys_helper_test.rb b/test/helpers/hotkeys_helper_test.rb
index ebade4aa5..ec96f6c9c 100644
--- a/test/helpers/hotkeys_helper_test.rb
+++ b/test/helpers/hotkeys_helper_test.rb
@@ -12,31 +12,31 @@ class HotkeysHelperTest < ActionView::TestCase
   test "linux modifier key" do
     emulate_linux
 
-    assert_equal "ctrlJ", hotkey_label([ "ctrl", "J" ])
+    assert_equal "Ctrl+J", hotkey_label([ "ctrl", "J" ])
   end
 
   test "mac enter" do
     emulate_mac
 
-    assert_equal "returnJ", hotkey_label([ "enter", "J" ])
+    assert_equal "Return+J", hotkey_label([ "enter", "J" ])
   end
 
   test "linux enter" do
     emulate_linux
 
-    assert_equal "enterJ", hotkey_label([ "enter", "J" ])
+    assert_equal "Enter+J", hotkey_label([ "enter", "J" ])
   end
 
   test "mac hyper" do
     emulate_mac
 
-    assert_equal "hyperJ", hotkey_label([ "hyper", "J" ])
+    assert_equal "Hyper+J", hotkey_label([ "hyper", "J" ])
   end
 
   test "linux hyper" do
     emulate_linux
 
-    assert_equal "hyperJ", hotkey_label([ "hyper", "J" ])
+    assert_equal "Hyper+J", hotkey_label([ "hyper", "J" ])
   end
 
   private