Skip to content

Conversation

@alexcarpenter
Copy link
Member

@alexcarpenter alexcarpenter commented Nov 14, 2025

Description

BEFORE AFTER
Screenshot 2025-11-14 at 1 44 04 PM Screenshot 2025-11-14 at 1 43 54 PM

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:

Summary by CodeRabbit

  • Style
    • Improved the profile card layout to ensure the scrollable content area properly utilizes available vertical space.

@vercel
Copy link

vercel bot commented Nov 14, 2025

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

Project Deployment Preview Comments Updated (UTC)
clerk-js-sandbox Ready Ready Preview Comment Nov 14, 2025 7:59pm

@changeset-bot
Copy link

changeset-bot bot commented Nov 14, 2025

🦋 Changeset detected

Latest commit: b201c42

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

This PR includes changesets to release 3 packages
Name Type
@clerk/clerk-js Patch
@clerk/chrome-extension Patch
@clerk/clerk-expo 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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 14, 2025

Walkthrough

A changeset entry marks a patch release for @clerk/clerk-js alongside a single styling adjustment that adds flex: 1 to the ProfileCard outer container, enabling the scrollbox to dynamically fill available vertical space.

Changes

Cohort / File(s) Summary
Release Management
.changeset/proud-months-go.md
Changeset entry documenting patch release notes for @clerk/clerk-js
Component Styling
packages/clerk-js/src/ui/elements/ProfileCard/ProfileCardContent.tsx
Added flex: 1 to outer container sx style to enable scrollbox to fill available vertical space

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Minimal scope: single style property addition and metadata file
  • No logic, control flow, or API changes
  • Straightforward layout adjustment with low risk of unintended side effects

Poem

🐰 A scrollbox that stretches with grace,
Now fills every bit of its space,
Flex one takes the lead,
Doing all that we need,
A patch for a prettier place! 📦✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title directly describes the main change: adding flex: 1 styling to ensure the scrollbox fills available height in ProfileCard component.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch alexcarpenter/fix-scrollbox-height

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 14, 2025

Open in StackBlitz

@clerk/agent-toolkit

npm i https://pkg.pr.new/@clerk/agent-toolkit@7222

@clerk/astro

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

@clerk/backend

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

@clerk/chrome-extension

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

@clerk/clerk-js

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

@clerk/dev-cli

npm i https://pkg.pr.new/@clerk/dev-cli@7222

@clerk/elements

npm i https://pkg.pr.new/@clerk/elements@7222

@clerk/clerk-expo

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

@clerk/expo-passkeys

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

@clerk/express

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

@clerk/fastify

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

@clerk/localizations

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

@clerk/nextjs

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

@clerk/nuxt

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

@clerk/clerk-react

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

@clerk/react-router

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

@clerk/remix

npm i https://pkg.pr.new/@clerk/remix@7222

@clerk/shared

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

@clerk/tanstack-react-start

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

@clerk/testing

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

@clerk/themes

npm i https://pkg.pr.new/@clerk/themes@7222

@clerk/types

npm i https://pkg.pr.new/@clerk/types@7222

@clerk/upgrade

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

@clerk/vue

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

commit: b201c42

Copy link

@barvian barvian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙌 fixes it on dashboard at least

@alexcarpenter alexcarpenter merged commit 4d7a6e0 into main Nov 14, 2025
43 checks passed
@alexcarpenter alexcarpenter deleted the alexcarpenter/fix-scrollbox-height branch November 14, 2025 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants