Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

Wumpf
Copy link
Member

@Wumpf Wumpf commented Aug 15, 2025

Related

What

Command palette (cmd/ctrl + P) can now import links!
(didn't untangle those two into separate PRs because I ran out of time, sorry!)

Command palette is sensitive to what kind of link you provide:

command.palette.knows.too.much.mov

@Wumpf Wumpf added 📺 re_viewer affects re_viewer itself do-not-merge Do not merge this PR include in changelog labels Aug 15, 2025
Copy link

github-actions bot commented Aug 15, 2025

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

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

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

@Wumpf Wumpf changed the title Open links from command palette, support opening web viewer links Open any importable link from command palette Aug 15, 2025
@Wumpf Wumpf force-pushed the andreas/url-imports branch from cd94aac to a79318b Compare August 15, 2025 16:45
Base automatically changed from andreas/fix-datasource-link-handling to main August 16, 2025 07:26
@Wumpf Wumpf force-pushed the andreas/url-imports branch from a79318b to 83b1b56 Compare August 16, 2025 07:29
Copy link
Member

@abey79 abey79 left a comment

Choose a reason for hiding this comment

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

Noice!

Comment on lines 137 to 138
// All of these send commands, make sure they'll be processed.
egui_ctx.request_repaint();
Copy link
Member

Choose a reason for hiding this comment

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

The fact that the system command queue is not empty at the end of a frame should automatically trigger a request repaint, so we don't have to think of it here.

@Wumpf Wumpf force-pushed the andreas/url-imports branch from 83b1b56 to 481a70b Compare August 25, 2025 07:44
@Wumpf Wumpf removed the do-not-merge Do not merge this PR label Aug 25, 2025
@Wumpf Wumpf merged commit de0cfee into main Aug 25, 2025
39 checks passed
@Wumpf Wumpf deleted the andreas/url-imports branch August 25, 2025 08:30
@aedm aedm added exclude from changelog PRs with this won't show up in CHANGELOG.md and removed include in changelog labels Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exclude from changelog PRs with this won't show up in CHANGELOG.md 📺 re_viewer affects re_viewer itself
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants