fix: show platform-specific shortcut labels in toolbar tooltips - #135
fix: show platform-specific shortcut labels in toolbar tooltips#135shrutig-blip wants to merge 1 commit into
Conversation
|
@shrutig-blip is attempting to deploy a commit to the rinava's projects Team on Vercel. A member of the Team first needs to authorize it. |
e5d3ffc to
abbeea4
Compare
|
@shrutig-blip rather than leave this stranded behind the Prettier landing, I took it over the line myself — rebuilt on today's main so the diff is just the fix. Your July requote is what's now on main, which is why the old branch had gone all-conflicts; the reconstruction drops that noise and keeps only the label change (16/−12, still under your authorship). What's in it:
One deliberate departure from your version: I left the Verified locally and in CI — Heads-up: I force-pushed onto your branch, so |
Summary
Fixes platform-specific shortcut labels in the Markdown toolbar tooltips.
The keyboard shortcut handler already detects the user's platform and uses
Ctrlon Windows/Linux and⌘on macOS, but the toolbar tooltips always displayed macOS shortcuts. This change derives the tooltip labels from the same platform check so they accurately reflect the shortcuts available on the current OS.Changes
macOSvsWindows/Linux).⌘in tooltip labels on macOS.Ctrlin tooltip labels on Windows/Linux.Closes #41