Skip to content

link_axis and link_cursor take an Id - #266

Merged
emilk merged 1 commit into
mainfrom
emilk/link-group-id
Sep 18, 2026
Merged

emilk merged 1 commit into
mainfrom
emilk/link-group-id

Conversation

@emilk

@emilk emilk commented Sep 18, 2026

Copy link
Copy Markdown
Owner

Replaces the last two impl Into<Id> parameters with plain Id, matching Plot::id and the earlier impl Into<f32> cleanup in:

Also documents that the group id must be globally unique, and that the very same id must be passed to each plot you want linked. Both link groups live in a single app-wide store keyed by that id, so a colliding id silently links unrelated plots.

  • I have followed the instructions in the PR template

🤖 Generated with Claude Code

Also document that the group id must be globally unique, and that the very
same id must be passed to each plot in the group. The link groups all live in
one app-wide store, so a colliding id silently links unrelated plots.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@emilk emilk added the include in changelog This change will be included in the changelog label Sep 18, 2026
@github-actions

Copy link
Copy Markdown

View snapshot changes at kitdiff

@emilk
emilk marked this pull request as ready for review September 18, 2026 09:15
@emilk
emilk merged commit 4cbd4a2 into main Sep 18, 2026
12 checks passed
@emilk
emilk deleted the emilk/link-group-id branch September 18, 2026 09:15
@emilk emilk mentioned this pull request Sep 18, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

include in changelog This change will be included in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant