Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Keyboard shortcuts cheat-sheet #452

Merged
merged 3 commits into from
Oct 12, 2023
Merged

Keyboard shortcuts cheat-sheet #452

merged 3 commits into from
Oct 12, 2023

Conversation

PaulHax
Copy link
Collaborator

@PaulHax PaulHax commented Oct 10, 2023

Pressing ? or clicking button on setting menu pops a modal:

image

@netlify
Copy link

netlify bot commented Oct 10, 2023

Deploy Preview for volview-dev ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 2c611a2
🔍 Latest deploy log https://app.netlify.com/sites/volview-dev/deploys/6525b5ff4855330008f5a82b
😎 Deploy Preview https://deploy-preview-452--volview-dev.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@PaulHax PaulHax changed the title Keyboard shortcuts cheetsheet Keyboard shortcuts cheet-sheet Oct 10, 2023
@floryst
Copy link
Collaborator

floryst commented Oct 10, 2023

Looks good! Two minor comments:

  • Using esc to close the popup would be nice
  • Let's try a monospace font for the letters. I think that would look more consistent.

@PaulHax PaulHax changed the title Keyboard shortcuts cheet-sheet Keyboard shortcuts cheat-sheet Oct 10, 2023
Lets dialogs of dialogs close on Escape key.
@PaulHax
Copy link
Collaborator Author

PaulHax commented Oct 10, 2023

Fixed the Escape key and added monospacing on the keybindings column.

@floryst floryst merged commit 1a82091 into Kitware:main Oct 12, 2023
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