Skip to content

fix: pass zoom param from embed URL to chart components#4350

Draft
SylvainChevalier wants to merge 1 commit intomainfrom
claude/issue-4332-the-x-axis-zoom-selector
Draft

fix: pass zoom param from embed URL to chart components#4350
SylvainChevalier wants to merge 1 commit intomainfrom
claude/issue-4332-the-x-axis-zoom-selector

Conversation

@SylvainChevalier
Copy link
Contributor

The embed modal correctly adds a zoom URL parameter to the iframe URL, but the embed page was not reading this parameter and passing it down to the chart components. This caused the zoom dropdown in the embed modal to have no effect on the rendered chart.

Fixes #4332

Generated with Claude Code

The embed modal correctly adds a zoom URL parameter to the iframe URL, but the embed page was not reading this parameter and passing it down to the chart components. This caused the zoom dropdown in the embed modal to have no effect on the rendered chart.

Fixes #4332

Co-authored-by: Sylvain <SylvainChevalier@users.noreply.github.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 13, 2026

Important

Review skipped

Draft detected.

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.

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
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/issue-4332-the-x-axis-zoom-selector

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.

@github-actions
Copy link
Contributor

🚀 Preview Environment

Your preview environment is ready!

Resource Details
🌐 Preview URL https://metaculus-pr-4350-claude-issue-4332-the-x-axis-z-preview.mtcl.cc
📦 Docker Image ghcr.io/metaculus/metaculus:claude-issue-4332-the-x-axis-zoom-selector-c8bcb3f
🗄️ PostgreSQL NeonDB branch preview/pr-4350-claude-issue-4332-the-x-axis-z
Redis Fly Redis mtc-redis-pr-4350-claude-issue-4332-the-x-axis-z

Details

  • Commit: fae51e12fb612c98ee3c754f54dc47c0487dfefb
  • Branch: claude/issue-4332-the-x-axis-zoom-selector
  • Fly App: metaculus-pr-4350-claude-issue-4332-the-x-axis-z

ℹ️ Preview Environment Info

Isolation:

  • PostgreSQL and Redis are fully isolated from production
  • Each PR gets its own database branch and Redis instance
  • Changes pushed to this PR will trigger a new deployment

Limitations:

  • Background workers and cron jobs are not deployed in preview environments
  • If you need to test background jobs, use Heroku staging environments

Cleanup:

  • This preview will be automatically destroyed when the PR is closed

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.

The x axis zoom selector in the embed modal doesn't work

1 participant