Skip to content

fix: force 500px height in small iframes (nteract)#682

Merged
paddymul merged 1 commit intomainfrom
fix/nteract-iframe-height
Apr 4, 2026
Merged

fix: force 500px height in small iframes (nteract)#682
paddymul merged 1 commit intomainfrom
fix/nteract-iframe-height

Conversation

@paddymul
Copy link
Copy Markdown
Collaborator

Summary

  • When rendered inside a small iframe (e.g. nteract's isolated output frames), window.innerHeight is tiny (~115px), so dfvHeight = innerHeight/2 ≈ 57px — only the header row is visible
  • Now detects iframe + small viewport (< 300px) and forces a fixed 500px height, same pattern used for Google Colab and VSCode
  • Also fixes a CSS class name mismatch: JS was emitting inIframe but the CSS selectors target .in-iframe — so the iframe-specific min-height rules never applied

Test plan

  • Install from this branch in nteract and verify the widget renders at ~500px height
  • Verify Jupyter/JupyterLab rendering is unaffected (not in an iframe, so no change)
  • Verify Colab/VSCode still work (separate code path, unchanged)

🤖 Generated with Claude Code

When the widget is in an iframe with a small viewport (< 300px), use a
fixed 500px height like Colab/VSCode, instead of window.innerHeight/2
which yields a tiny widget. Also fix CSS class mismatch: JS was emitting
"inIframe" but CSS targets ".in-iframe".

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

📦 TestPyPI package published

pip install --index-strategy unsafe-best-match --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ buckaroo==0.13.2.dev23683363426

or with uv:

uv pip install --index-strategy unsafe-best-match --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ buckaroo==0.13.2.dev23683363426

MCP server for Claude Code

claude mcp add buckaroo-table -- uvx --from "buckaroo[mcp]==0.13.2.dev23683363426" --index-strategy unsafe-best-match --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ buckaroo-table

📖 Docs preview

🎨 Storybook preview

@paddymul paddymul added this pull request to the merge queue Apr 4, 2026
Merged via the queue into main with commit a2c2b83 Apr 4, 2026
27 checks passed
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