Refactor keyboard shortcuts to use uppercase keys for display and update KeyBadge functionality

This commit is contained in:
Anwarul Islam
2025-07-18 06:22:49 +06:00
parent f5d221feec
commit c391eb8616
2 changed files with 31 additions and 34 deletions
@@ -1,5 +1,7 @@
"use client";
import { useKeyboardShortcuts } from "@/hooks/use-keyboard-shortcuts";
import { Keyboard } from "lucide-react";
import { useState } from "react";
import { Button } from "./ui/button";
import {
@@ -10,38 +12,33 @@ import {
DialogTitle,
DialogTrigger,
} from "./ui/dialog";
import { Badge } from "./ui/badge";
import { Keyboard } from "lucide-react";
import { useKeyboardShortcuts } from "@/hooks/use-keyboard-shortcuts";
import { getPlatformSpecialKey } from "@/lib/utils";
const KeyBadge = ({ keyName }: { keyName: string }) => {
// Replace common key names with symbols or friendly names
return keyName
.replace("Cmd", "⌘")
.replace("Shift", "Shift")
.replace("ArrowLeft", "Arrow Left")
.replace("ArrowRight", "Arrow Right")
.replace("ArrowUp", "Arrow Up")
.replace("ArrowDown", "Arrow Down")
.replace("←", "◀")
.replace("→", "▶")
.replace("Space", "Space");
const modifier: {
[key: string]: string;
} = {
Shift: "Shift",
Alt: "Alt",
ArrowLeft: "←",
ArrowRight: "→",
ArrowUp: "↑",
ArrowDown: "↓",
Space: "Space",
};
function getKeyWithModifier(key: string) {
if (key === "Ctrl") return getPlatformSpecialKey();
return modifier[key] || key;
}
const ShortcutItem = ({ shortcut }: { shortcut: any }) => {
// Filter out lowercase duplicates for display - if both "j" and "J" exist, only show "J"
const displayKeys = shortcut.keys.filter((key: string) => {
const lowerKey = key.toLowerCase();
const upperKey = key.toUpperCase();
// If this is a lowercase letter and the uppercase version exists, skip it
if (
key === lowerKey &&
key !== upperKey &&
shortcut.keys.includes(upperKey)
) {
key.includes("Cmd") &&
shortcut.keys.includes(key.replace("Cmd", "Ctrl"))
)
return false;
}
return true;
});
@@ -60,7 +57,7 @@ const ShortcutItem = ({ shortcut }: { shortcut: any }) => {
<div className="flex items-center">
{key.split("+").map((keyPart: string, partIndex: number) => (
<kbd key={partIndex} className="shortcut-key">
<KeyBadge keyName={keyPart} />
{getKeyWithModifier(keyPart)}
</kbd>
))}
</div>