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

Streamline the help view shortcuts with the shortcuts in the menu #9268

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

lucasmerlin
Copy link
Contributor

@lucasmerlin lucasmerlin commented Mar 12, 2025

Related

What

Improves on the new help view, making the shortcuts look more like the shortcuts shown in a menu or command palette.
Also adds scresnshot tests for each help view, since they look pretty different on windows and mac.

Copy link

github-actions bot commented Mar 12, 2025

Web viewer built successfully. If applicable, you should also test it:

  • I have tested the web viewer
Result Commit Link Manifest
3f75f17 https://rerun.io/viewer/pr/9268 +nightly +main

Note: This comment is updated whenever you push a commit.

@lucasmerlin lucasmerlin added ui concerns graphical user interface enhancement New feature or request include in changelog labels Mar 12, 2025
@lucasmerlin lucasmerlin force-pushed the lucas/streamline-help-view-shortcuts branch from 174ec47 to 0c80752 Compare March 12, 2025 16:58
@lucasmerlin lucasmerlin marked this pull request as ready for review March 12, 2025 17:52
@Wumpf
Copy link
Member

Wumpf commented Mar 12, 2025

screenshot plz :)

@lucasmerlin
Copy link
Contributor Author

Mac:

image

Windows:

image

There is also snapshot tests for each now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request include in changelog ui concerns graphical user interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Streamline shortcut indicators between rerun menu, command palette and help view
2 participants