fix(CardHorizontal): prevent duplicate button clicks

[fallintoplace]

What changed

• Stop the inner button click from bubbling to the card
• The inner info button no longer triggers the card public onClick handler, including when onClick is passed through ...props
• Add tests for onButtonClick and window.open call counts
• Add a patch changeset

Why

When a CardHorizontal has an info button, clicking it also triggers the card handler. This could call the callback and open the URL twice.

Checks

• corepack yarn test --run
• corepack yarn typecheck
• corepack yarn format
• corepack yarn lint:code

[changeset-bot]

🦋 Changeset detected

Latest commit: b12e986

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@clickhouse/click-ui	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[Copilot]

🟢 Approval recommended

The event-propagation fix is targeted and is backed by regression tests covering the reported double-invocation behavior.

Pull request overview

This PR prevents duplicate click handling in CardHorizontal when the nested info button is present by stopping event bubbling from the button to the card container, and adds regression tests plus a patch changeset.

Changes:

• Add a dedicated handleButtonClick that calls stopPropagation() before delegating to the existing click handler.
• Add tests asserting onButtonClick and window.open are each invoked exactly once when clicking the inner button.
• Add a patch changeset documenting the fix.

File summaries

File	Description
src/components/CardHorizontal/CardHorizontal.tsx	Stops nested button click events from bubbling to the card, preventing duplicate handling.
src/components/CardHorizontal/CardHorizontal.test.tsx	Adds regression tests for single-invocation behavior on inner button clicks.
.changeset/fix-card-horizontal-button-bubbling.md	Records the fix as a patch release entry.

Review details

• Files reviewed: 3/3 changed files
• Comments generated: 0
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[XOP]

Hi @fallintoplace , thank you for contribution!

LGTM,
One thing that I would like to add to the changes - a note about clicking the inner button - the card exposes onClick publicly, and {...props} spreads after the internal handler, so consumers who wired it used to get button clicks too. Now they don't.

[XOP]

note: this will close #998