Skip to content

feat(ui): ConfirmButton size + cancelSide props - #15

Closed
jcfrei wants to merge 1 commit into
masterfrom
feat/confirm-button-props
Closed

feat(ui): ConfirmButton size + cancelSide props#15
jcfrei wants to merge 1 commit into
masterfrom
feat/confirm-button-props

Conversation

@jcfrei

@jcfrei jcfrei commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Adds two opt-in props to ConfirmButton so a right-aligned trigger (e.g. a toolbar delete button) is double-click-safe:

  • cancelSide?: "left" | "right" (default "left") — a right-aligned armed group keeps its right edge, so Cancel must be on the right to sit under the trigger's original click point. Left-aligned triggers keep the default.
  • size?: "sm" | "md" (default "sm") — "md" renders normal-height armed buttons to match a full-size labelled trigger; "sm" keeps the compact style for icon triggers.

Defaults are unchanged → existing usages (icon trashes) render byte-identical. The consumer (internal CRM lead delete) will pass size="md" cancelSide="right".

🤖 Generated with Claude Code

…-click safety)

For a right-aligned trigger the armed group keeps its right edge, so Cancel must
sit on the right to land under a double-click — add cancelSide ("left"|"right").
size ("sm"|"md") sizes the armed buttons to match a full-size labelled trigger.
Defaults preserve current rendering.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
jcfrei added a commit that referenced this pull request Jul 30, 2026
feat(ui): ConfirmButton size + cancelSide props (#15)

Right-aligned triggers keep their right edge when armed, so Cancel goes on the
right (cancelSide) to be double-click-safe; size="md" matches a full-size
trigger. Defaults unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@jcfrei

jcfrei commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

Released as v0.6.11.

@jcfrei jcfrei closed this Jul 30, 2026
@jcfrei
jcfrei deleted the feat/confirm-button-props branch July 30, 2026 17:55
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