Skip to content

fix(profile-form): remove unnecessary styling from FieldDescription#139

Merged
chef0111 merged 1 commit intomainfrom
staging
Feb 2, 2026
Merged

fix(profile-form): remove unnecessary styling from FieldDescription#139
chef0111 merged 1 commit intomainfrom
staging

Conversation

@chef0111
Copy link
Copy Markdown
Owner

@chef0111 chef0111 commented Feb 2, 2026

…omponents

Summary by CodeRabbit

  • Style
    • Refined field description styling on profile page
  • New Features
    • Admin dashboard link now opens in a new tab
  • Changes
    • Updated loading state display for user navigation

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 2, 2026

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

Project Deployment Actions Updated (UTC)
dev4room Building Building Preview, Comment Feb 2, 2026 9:32am

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 2, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request applies three UI refinements: removing light text styling from profile loading placeholders, configuring the admin dashboard link to open in a new tab, and replacing the skeleton loading state with null rendering in the sidebar user component.

Changes

Cohort / File(s) Summary
Profile Loading Styling
src/app/(root)/(account)/settings/profile/loading.tsx
Removed text-light-500 class from 5 FieldDescription elements, retaining only body-regular styling.
Navigation Updates
src/components/modules/profile/user-nav.tsx
Added target="_blank" attribute to AdminItem link to open Dashboard in a new browser tab.
Sidebar Pending State
src/components/navigation/left-sidebar/sidebar-user.tsx
Removed Skeleton import and replaced pending-state rendering with null instead of Skeleton placeholder; UserNav continues rendering when user exists.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • PR #86: Modifies left-sidebar SidebarUser component rendering logic, specifically adjusting Skeleton and pending state behavior similar to these changes.
  • PR #87: Updates the same SidebarUser component with changes to imports and conditional rendering around user state management.

Poem

🐰 Five fields shed their dimming gray,
A link hops open—new tab way,
Where Skeletons once danced with care,
Null whispers softly in the air.
Polish complete, the UI's bright! ✨

✨ 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 staging

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copy link
Copy Markdown
Owner Author

@chef0111 chef0111 left a comment

Choose a reason for hiding this comment

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

Approve

@chef0111 chef0111 merged commit 0e72256 into main Feb 2, 2026
3 of 6 checks passed
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