Skip to content

🤖 fix: improve heartbeat modal layout#3612

Merged
ThomasK33 merged 1 commit into
mainfrom
heartbeat-8aw6
Jun 23, 2026
Merged

🤖 fix: improve heartbeat modal layout#3612
ThomasK33 merged 1 commit into
mainfrom
heartbeat-8aw6

Conversation

@ThomasK33

@ThomasK33 ThomasK33 commented Jun 23, 2026

Copy link
Copy Markdown
Member

Summary

Improves the workspace heartbeat configuration modal so long custom messages are easier to read and edit. The dialog now expands wider on desktop, uses a two-column layout when heartbeats are enabled, keeps actions pinned in a footer, and adds desktop/mobile Storybook scenarios for the long-message layout.

Background

The previous modal constrained the heartbeat message to a narrow field, making recurring background prompts hard to review before saving.

Implementation

  • Widened the heartbeat modal at larger breakpoints while preserving a constrained mobile width.
  • Split heartbeat controls and the message editor into separate cards on desktop.
  • Added a scrollable content region with a pinned footer for Save/Cancel actions.
  • Added Storybook coverage for long-message desktop and mobile layouts.

Validation

  • make static-check
  • bun test src/browser/components/WorkspaceHeartbeatModal/WorkspaceHeartbeatModal.test.tsx
  • make typecheck
  • MUX_ESLINT_CONCURRENCY=1 make lint
  • make fmt-check
  • Dogfooded in Storybook with agent-browser at 1280×800 and 375×667, capturing screenshots and a WebM recording.

Risks

Low. This is scoped to the heartbeat settings modal presentation and Storybook coverage; existing save/load behavior is covered by the focused modal tests.


Generated with mux • Model: openai:gpt-5.5 • Thinking: xhigh • Cost: $12.69

Make the workspace heartbeat settings dialog wider on desktop, split the message editor into a larger column, and add Storybook coverage for desktop/mobile long-message layouts.

---
_Generated with `mux` • Model: `openai:gpt-5.5` • Thinking: `xhigh` • Cost: `1250988{MUX_COSTS_USD:-unknown}`_

<!-- mux-attribution: model=openai:gpt-5.5 thinking=xhigh costs=12.69 -->
@ThomasK33

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

Reviewed commit: 1411df59fb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ThomasK33 ThomasK33 added this pull request to the merge queue Jun 23, 2026
Merged via the queue into main with commit 2ab5a3f Jun 23, 2026
38 of 41 checks passed
@ThomasK33 ThomasK33 deleted the heartbeat-8aw6 branch June 23, 2026 15:15
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