Skip to content

fix(ui): tighten resource header action spacing - #7172

Merged
waleedlatif1 merged 1 commit into
stagingfrom
codex/fix-resource-header-action-spacing
Aug 27, 2026
Merged

fix(ui): tighten resource header action spacing#7172
waleedlatif1 merged 1 commit into
stagingfrom
codex/fix-resource-header-action-spacing

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Move resource header actions 12px right while preserving the collapse toggle position and 40px hit target
  • Add a focused geometry regression test

Type of Change

  • Bug fix

Testing

  • bun run lint
  • bun run lint:check
  • bun run check:audits
  • bun run apps/sim/scripts/check-block-registry.ts origin/staging
  • Resource tab unit tests (4 passing)

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 27, 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 27, 2026 6:58pm

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 2 files

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 27, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR tightens resource-header action spacing by reducing the reserved end space from 64px to 52px while retaining the collapse toggle’s position and 40px hit target.

  • Updates the shared resource-header geometry token.
  • Adds focused assertions covering the inset, reserve, toggle size, hit-target size, and end positioning.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete regressions identified in the changed header geometry.

The new 52px reserve remains outside the centered 40px hit target of the 30px toggle at its unchanged 16px inset, and the accompanying test locks the intended shared tokens.

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-tabs/resource-tab-controls.ts Reduces the fixed end reserve to move header actions 12px right while preserving separation from the collapse-toggle hit target.
apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-tabs/resource-tab-controls.test.ts Adds focused regression assertions for the shared resource-header geometry constants.

Reviews (1): Last reviewed commit: "fix(ui): tighten resource header action ..." | Re-trigger Greptile

@waleedlatif1
waleedlatif1 merged commit 9efbe36 into staging Aug 27, 2026
30 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/fix-resource-header-action-spacing branch August 27, 2026 19:47
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