Skip to content

feat(swingset): Add Tooltip primitive#8807

Closed
kylemac wants to merge 1 commit into
mainfrom
kylemac/swingset-tooltip-primitive
Closed

feat(swingset): Add Tooltip primitive#8807
kylemac wants to merge 1 commit into
mainfrom
kylemac/swingset-tooltip-primitive

Conversation

@kylemac

@kylemac kylemac commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Description

Adds a self-contained Mosaic Tooltip component (@clerk/ui/mosaic) that reveals a label on hover or keyboard focus, with a side variant for placement, and documents it in the swingset component explorer via an interactive "Options" story plus MDX docs. While testing the freshly scaffolded swingset app, this also fixes three latent bugs: the StoryCanvas knob-reset effect that looped infinitely (it depended on a findStory object that was new every render — now derived via useMemo on the slug primitives), the sidebar Collapsible uncontrolled-state warning (now a controlled per-group component that opens on the active route), and the dev-only <Script> in the root layout rendering outside <head>. To test: pnpm dev --filter @clerk/swingset and open /components/tooltip, then hover/focus the trigger buttons. No published API surface changes (swingset is private and Mosaic isn't yet exported from @clerk/ui), so the changeset is empty.

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@kylemac
feat(swingset): Add Tooltip primitive and fix canvas/sidebar bugs
9a41890 
Adds a self-contained Mosaic Tooltip component and documents it in swingset
with an interactive "Options" story showing all four sides. Also fixes a few
latent bugs in the freshly scaffolded swingset app surfaced while testing:
the StoryCanvas knob-reset effect that looped infinitely, the sidebar
Collapsible uncontrolled-state warning, and the dev-only Script placement in
the root layout.
@vercel

vercel Bot commented Jun 10, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clerk-js-sandbox Ready Ready Preview, Comment Jun 10, 2026 6:03pm

Request Review

@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository YAML (base), Repository UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 4cba4de6-4b63-42a0-98b0-a74aded693b6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added the ui label Jun 10, 2026
@changeset-bot

changeset-bot Bot commented Jun 10, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9a41890

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

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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

@pkg-pr-new

pkg-pr-new Bot commented Jun 10, 2026

Copy link
Copy Markdown

Open in StackBlitz

@clerk/astro

npm i https://pkg.pr.new/@clerk/astro@8807

@clerk/backend

npm i https://pkg.pr.new/@clerk/backend@8807

@clerk/chrome-extension

npm i https://pkg.pr.new/@clerk/chrome-extension@8807

@clerk/clerk-js

npm i https://pkg.pr.new/@clerk/clerk-js@8807

@clerk/expo

npm i https://pkg.pr.new/@clerk/expo@8807

@clerk/expo-passkeys

npm i https://pkg.pr.new/@clerk/expo-passkeys@8807

@clerk/express

npm i https://pkg.pr.new/@clerk/express@8807

@clerk/fastify

npm i https://pkg.pr.new/@clerk/fastify@8807

@clerk/hono

npm i https://pkg.pr.new/@clerk/hono@8807

@clerk/localizations

npm i https://pkg.pr.new/@clerk/localizations@8807

@clerk/nextjs

npm i https://pkg.pr.new/@clerk/nextjs@8807

@clerk/nuxt

npm i https://pkg.pr.new/@clerk/nuxt@8807

@clerk/react

npm i https://pkg.pr.new/@clerk/react@8807

@clerk/react-router

npm i https://pkg.pr.new/@clerk/react-router@8807

@clerk/shared

npm i https://pkg.pr.new/@clerk/shared@8807

@clerk/tanstack-react-start

npm i https://pkg.pr.new/@clerk/tanstack-react-start@8807

@clerk/testing

npm i https://pkg.pr.new/@clerk/testing@8807

@clerk/ui

npm i https://pkg.pr.new/@clerk/ui@8807

@clerk/upgrade

npm i https://pkg.pr.new/@clerk/upgrade@8807

@clerk/vue

npm i https://pkg.pr.new/@clerk/vue@8807

commit: 9a41890

@kylemac kylemac closed this Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

ui

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kylemac