Skip to content

fix(logs): prevent dashboard horizontal scrolling - #7223

Merged
waleedlatif1 merged 1 commit into
stagingfrom
codex/fix-logs-dashboard-overflow
Aug 28, 2026
Merged

fix(logs): prevent dashboard horizontal scrolling#7223
waleedlatif1 merged 1 commit into
stagingfrom
codex/fix-logs-dashboard-overflow

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Prevent the dashboard scrollport from including the off-canvas log details panel
  • Preserve vertical dashboard scrolling while clipping unintended horizontal overflow

Type of Change

  • Bug fix

Testing

  • bun run lint:check
  • bun run lint
  • bun run apps/sim/scripts/check-block-registry.ts origin/staging
  • bun run check:audits

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Aug 28, 2026 8:06pm

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR prevents the logs dashboard’s off-canvas details panel from expanding the horizontal scroll area while preserving vertical scrolling.

  • Changes the outer dashboard container from an automatic scrollport to a clipping boundary.
  • Moves vertical scrolling to the inner content container and constrains its width.
  • Explicitly clips unintended horizontal overflow.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The new inner container preserves vertical scrolling and width contraction, while the outer clipping boundary excludes the absolutely positioned details panel from the dashboard’s horizontal scroll area.

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/logs/logs.tsx Adjusts nested dashboard overflow ownership to prevent horizontal page scrolling without introducing a concrete regression.

Reviews (1): Last reviewed commit: "fix(logs): prevent dashboard horizontal ..." | Re-trigger Greptile

@waleedlatif1
waleedlatif1 merged commit 2bda859 into staging Aug 28, 2026
21 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/fix-logs-dashboard-overflow branch August 28, 2026 20:12
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