Skip to content

fix-r-return-to-city#78

Open
gigacat-25 wants to merge 1 commit intosrizzon:mainfrom
gigacat-25:fix-r-return-to-city
Open

fix-r-return-to-city#78
gigacat-25 wants to merge 1 commit intosrizzon:mainfrom
gigacat-25:fix-r-return-to-city

Conversation

@gigacat-25
Copy link
Copy Markdown

What does this PR do?

Related issue

Screenshots

Checklist

  • npm run lint passes
  • Tested locally
  • No secrets or .env values committed

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 22, 2026

@gigacat-25 is attempting to deploy a commit to the Samuel's projects Team on Vercel.

A member of the Team first needs to authorize it.

@srizzon
Copy link
Copy Markdown
Owner

srizzon commented Mar 24, 2026

Hey @gigacat-25, this adds the "R" shortcut to the controls legend, but does the keybind itself already exist? I don't see the actual key handler being added in this PR — just the UI text. Let me know if I'm missing something.

Also the whitespace changes in the catch blocks (adding a space inside the braces) aren't needed — please keep those as-is to avoid unnecessary diff noise.

Copy link
Copy Markdown
Owner

@srizzon srizzon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed for security. Clean PR with only static string additions to the HUD controls overlay. No user input rendered, no new API calls, consistent with existing patterns. Safe to merge. Thanks for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants