fix(ui): Update org switcher avatar alignment and radii#9148
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🦋 Changeset detectedLatest commit: 5839078 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
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 |
📝 WalkthroughWalkthroughUpdated the organization switcher trigger’s padding pattern, applied a small border radius to organization avatars, and added a patch changeset. ChangesOrganization switcher styling
Estimated code review effort: 1 (Trivial) | ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
API Changes Report
Summary
No API Changes DetectedAll packages have stable APIs with no detected changes. Report generated by Break Check Last ran on |
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/electron
@clerk/electron-passkeys
@clerk/eslint-plugin
@clerk/expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/hono
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/react
@clerk/react-router
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/ui
@clerk/upgrade
@clerk/vue
commit: |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.changeset/org-switcher-trigger-avatar-radius.md:
- Line 5: Update the changeset release note for the OrganizationSwitcher trigger
to document both user-visible changes: the reduced avatar border radius and the
accompanying left-padding adjustment. Keep the existing avatar-radius
description and add the padding change without altering unrelated changeset
metadata.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Repository UI (inherited)
Review profile: CHILL
Plan: Pro Plus
Run ID: f3394a7a-e995-40be-93b4-1b06b2491868
📒 Files selected for processing (1)
.changeset/org-switcher-trigger-avatar-radius.md
| '@clerk/ui': patch | ||
| --- | ||
|
|
||
| Reduce the organization avatar's border radius in the `OrganizationSwitcher` trigger so it stays proportional at the smaller trigger size. |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Document the trigger padding change in the changeset.
The release note mentions the avatar-radius adjustment but omits the accompanying left-padding change. Please describe both user-visible updates so the @clerk/ui changelog accurately reflects this patch.
This follows the stated PR objective, which includes both trigger padding and avatar-radius changes.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In @.changeset/org-switcher-trigger-avatar-radius.md at line 5, Update the
changeset release note for the OrganizationSwitcher trigger to document both
user-visible changes: the reduced avatar border radius and the accompanying
left-padding adjustment. Keep the existing avatar-radius description and add the
padding change without altering unrelated changeset metadata.
Description
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change
Summary by CodeRabbit