Skip to content

fix: make sidebar integration errors accessible#4008

Open
forestileao wants to merge 5 commits intomainfrom
cursor/tippy-tooltip-integration-error-9eb0
Open

fix: make sidebar integration errors accessible#4008
forestileao wants to merge 5 commits intomainfrom
cursor/tippy-tooltip-integration-error-9eb0

Conversation

@forestileao
Copy link
Copy Markdown
Collaborator

@forestileao forestileao commented Apr 7, 2026

Summary

  • Fixes the integration connection error tooltip in the configuration sidebar by constraining the tooltip’s size (max width/height) so long error messages wrap and scroll.
  • Avoids forcing a large tooltip width for short messages by removing the fixed 90vw width.
  • Prevents tooltip regressions by making SimpleTooltip non-interactive by default; the integration error tooltip opts into interactivity + appendTo(document.body) so long content can be scrolled without being clipped.

Notes

  • make check.build.ui requires Docker in this environment; I validated the UI build with cd web_src && npm run build instead.
Open in Web Open in Cursor 

Co-authored-by: Pedro Leão <forestileao@users.noreply.github.com>
@superplanehq-integration
Copy link
Copy Markdown

👋 Commands for maintainers:

  • /sp start - Start an ephemeral machine (takes ~30s)
  • /sp stop - Stop a running machine (auto-executed on pr close)

@forestileao
Copy link
Copy Markdown
Collaborator Author

@cursoragent I think the fix was just a max-width, not all this stuff

@cursor
Copy link
Copy Markdown
Contributor

cursor bot commented Apr 7, 2026

What I did in response to the PR comment

  • Simplified the fix to match the review: I removed the new inline expandable “Connection error” UI and reverted SettingsTab back to the original tooltip-only behavior.
  • Kept the minimal, targeted improvement: SimpleTooltip now enforces a sane max width/height with wrapping + scrolling, so long integration error text is readable instead of a single huge line.

Verification + Git

View PR

Open in Web Open in Cursor 

Co-authored-by: Pedro Leão <forestileao@users.noreply.github.com>
@forestileao forestileao marked this pull request as ready for review April 7, 2026 17:11
Co-authored-by: Pedro Leão <forestileao@users.noreply.github.com>
@forestileao
Copy link
Copy Markdown
Collaborator Author

/sp start

@superplanehq-integration
Copy link
Copy Markdown

✅ Ready.

Web: https://pr-4008-ephemeral.superplane.com
SSH: ssh -o StrictHostKeyChecking=no app@178.104.15.210
Logs: ssh -o StrictHostKeyChecking=no app@178.104.15.210 'cd superplane && make dev.logs.app'

Co-authored-by: Pedro Leão <forestileao@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 7fdbded. Configure here.

Co-authored-by: Pedro Leão <forestileao@users.noreply.github.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.

2 participants