Skip to content

feat(dev-preview): convert restart button to icon-only in ConsoleDrawer#3422

Merged
gregpriday merged 2 commits intodevelopfrom
feature/issue-3418-dev-preview-restart-button
Mar 16, 2026
Merged

feat(dev-preview): convert restart button to icon-only in ConsoleDrawer#3422
gregpriday merged 2 commits intodevelopfrom
feature/issue-3418-dev-preview-restart-button

Conversation

@gregpriday
Copy link
Collaborator

Summary

  • Replaced the labeled <Button> restart control in ConsoleDrawer footer with a compact icon-only button matching the pattern used in DevPreviewToolbar
  • The tooltip already carried the full action label ("Hard restart dev preview"), so the visible text span was redundant
  • Spinning and disabled states during restart/start transitions continue to work correctly

Resolves #3418

Changes

  • src/components/DevPreview/ConsoleDrawer.tsx: swapped <Button> with a plain <button> element styled as an icon control; removed the <span>Restart</span> label
  • src/components/DevPreview/__tests__/ConsoleDrawer.test.tsx: added assertion confirming no visible "Restart" text label is rendered in the footer

Testing

Unit test added and passing. The restart button now renders icon-only, consistent with the toolbar restart button. Tooltip and state behaviour verified against the existing test suite.

- Replace shadcn Button with plain button element for icon-only styling
- Remove visible "Restart" text label, keeping only RotateCw icon
- Apply compact toolbar-consistent styles (p-1.5, rounded, hover:bg-overlay-medium)
- Move animate-spin from icon to button element matching DevPreviewToolbar pattern
- Remove unused Button import from @/components/ui/button
- Verify restart button has no visible text content
- Confirm SVG icon is present inside the button element
@gregpriday
Copy link
Collaborator Author

Review status: Ready

Cross-PR holistic review — no file overlap or semantic conflicts with other open PRs (#3421, #3423, #3424). All 4 PRs are independent and safe to merge in any order.

@gregpriday gregpriday merged commit 2b48db7 into develop Mar 16, 2026
4 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.

Dev preview restart button should be an icon, not a labeled button

1 participant