Skip to content

Mark point symbols#219

Open
samheuck wants to merge 8 commits into
yuankunzhang:mainfrom
samheuck:mark-point-symbols
Open

Mark point symbols#219
samheuck wants to merge 8 commits into
yuankunzhang:mainfrom
samheuck:mark-point-symbols

Conversation

@samheuck

Copy link
Copy Markdown
Contributor

Allows fine-tuning of label positioning and style

samheuck and others added 7 commits February 27, 2026 12:11
Signed-off-by: Sam Heuck <sam@emushack.com>
Signed-off-by: Sam Heuck <sam@emushack.com>
Signed-off-by: Sam Heuck <sam@emushack.com>
Remove explicit #[serde(skip_serializing_if)] on fields already covered
by #[serde_with::apply], and drop PartialOrd from MarkPoint since Symbol,
SymbolSize, and Label don't implement it.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Sam Heuck <sam@emushack.com>
@samheuck

Copy link
Copy Markdown
Contributor Author

CI is currently failing because: dioxus 0.6 pulls in wasm-bindgen 0.2.114 at resolve time. nixpkgs in the devenv lock is too old, providing only 0.2.104. The fix is to update the devenv to a newer nixpkgs snapshot that ships 0.2.114

I don't have a devenv setup for this repo at the moment, but I can try do do this locally and report back.

@samheuck

samheuck commented Feb 27, 2026

Copy link
Copy Markdown
Contributor Author

CI appears to be blocked by an upstream dependency on wasm-bindgen-cli 0.2.114

Ideally, NixOS/nixpkgs would pull in this update at some point. Watching their PR queue via search: https://github.com/NixOS/nixpkgs/pulls?q=is:pr+is:open+wasm-bindgen-cli

I'm wondering if there is an alternative? Could/Should it be included in this projects Cargo.toml file as a dependency? Even if temporary until the devenv gets updated?

Signed-off-by: Sam Heuck <sam@emushack.com>
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.

1 participant