Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork21
fix: improve automatic compact notation detection#278
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
📦 Preview package for this PR is published! Version: Install it with: npm install svelteplot@pr-278# or install the specific versionnpm install svelteplot@0.7.1-pr-278.0 |
📦 Preview package for this PR is published! Version: Install it with: npm install svelteplot@pr-278# or install the specific versionnpm install svelteplot@0.7.1-pr-278.1 |
netlifybot commentedNov 27, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
✅ Deploy Preview forsvelteplot ready!
To edit notification comments on pull requests, go to yourNetlify project configuration. |
📦 Preview package for this PR is published! Version: Install it with: npm install svelteplot@pr-278# or install the specific versionnpm install svelteplot@0.7.1-pr-278.2 |
📦 Preview package for this PR is published! Version: Install it with: npm install svelteplot@pr-278# or install the specific versionnpm install svelteplot@0.7.1-pr-278.3 |
📦 Preview package for this PR is published! Version: Install it with: npm install svelteplot@pr-278# or install the specific versionnpm install svelteplot@0.7.1-pr-278.4 |
8386138 intomainUh oh!
There was an error while loading.Please reload this page.
This pull request refactors how compact notation is determined for axis tick labels in both the
AxisX.svelteandAxisY.sveltecomponents. The logic for deciding when to use compact notation (e.g., "1K" for 1,000) is now extracted into a derived store for clarity and consistency, and the code for formatting tick labels is updated to use this new logic.Axis tick label formatting improvements:
$derivedstore calleduseCompactNotationin bothAxisX.svelteandAxisY.svelte, improving code readability and maintainability.[1][2]useCompactNotationstore instead of recalculating the logic inline, ensuring consistent behavior and reducing code duplication.[1][2]