Skip to content

fix: persist zoom state in line charts#2689

Merged
ghostdevv merged 2 commits intomainfrom
tooltip-regression
May 8, 2026
Merged

fix: persist zoom state in line charts#2689
ghostdevv merged 2 commits intomainfrom
tooltip-regression

Conversation

@graphieros
Copy link
Copy Markdown
Contributor

@graphieros graphieros commented May 8, 2026

follow-up to #2688

This fixes the unwanted re-rendering of the zoom component in line charts for downloads when the chart config is dynamically updated.

Test:

  • display a downloads chart (modal, or compare page)
  • use the zoom
  • hover the chart: zoom does not reset

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment May 8, 2026 3:33pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview May 8, 2026 3:33pm
npmx-lunaria Ignored Ignored May 8, 2026 3:33pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 8, 2026

Review Change Stack

Important

Review skipped

Auto reviews are limited based on label configuration.

🚫 Review skipped — only excluded labels are configured. (1)
  • release

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 4c4da8a0-c4f4-4355-a47c-34f6b8c4977d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

The TrendsChart.vue component now preserves zoom state across chart updates by enabling keepState: true in the VueUiXy chart's zoom configuration options.

Changes

Zoom State Persistence

Layer / File(s) Summary
Chart Zoom Configuration
app/components/Package/TrendsChart.vue
VueUiXy zoom options updated to add keepState: true alongside existing zoom settings (maxWidth, highlightColor, useResetSlot).

Suggested reviewers

  • userquin
  • gameroman
  • MatteoGabriele
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: enabling zoom state persistence in line charts through a configuration update.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The pull request description clearly explains the purpose of the change: fixing unwanted re-rendering of the zoom component by preserving zoom state.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch tooltip-regression

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@graphieros graphieros changed the title fix: add missing config attr to keep zoom state fix: persist zoom state in line charts May 8, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@graphieros graphieros added the needs review This PR is waiting for a review from a maintainer label May 8, 2026
@ghostdevv ghostdevv added this pull request to the merge queue May 8, 2026
Merged via the queue into main with commit f8cff4c May 8, 2026
29 checks passed
@ghostdevv ghostdevv deleted the tooltip-regression branch May 8, 2026 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs review This PR is waiting for a review from a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants