Skip to content

chore(skeleton): use border-border token instead of hardcoded color - #243

Merged
Benjtalkshow merged 1 commit into
boundlessfi:mainfrom
Benjtalkshow:fix/skeleton-border-token
May 28, 2026
Merged

chore(skeleton): use border-border token instead of hardcoded color#243
Benjtalkshow merged 1 commit into
boundlessfi:mainfrom
Benjtalkshow:fix/skeleton-border-token

Conversation

@Benjtalkshow

@Benjtalkshow Benjtalkshow commented May 28, 2026

Copy link
Copy Markdown
Contributor

Replaces border-[#f0f0f0] dark:border-slate-700 with the standard border-border token in BountyCardSkeleton. Pre-existing from the old skeleton file, carried over during #236.

Summary by CodeRabbit

  • Style
    • Updated bounty card loading skeleton styling to use consistent theme-based colors, improving visual coherence across light and dark modes.

Review Change Stack

@vercel

vercel Bot commented May 28, 2026

Copy link
Copy Markdown

@Benjtalkshow is attempting to deploy a commit to the Threadflow Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented May 28, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 74f8ccd0-0f30-4379-a7a6-9c913b0a84e3

📥 Commits

Reviewing files that changed from the base of the PR and between eab87ee and d94905d.

📒 Files selected for processing (1)
  • components/ui/loading/bounty-card-skeleton.tsx

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Walkthrough

BountyCardSkeleton's CardFooter border styling was updated from hardcoded hex and dark slate color classes to a theme token-based class for design system consistency. No structural or API changes.

Changes

Theme Token Styling

Layer / File(s) Summary
BountyCardSkeleton border theme tokenization
components/ui/loading/bounty-card-skeleton.tsx
CardFooter border styling switched from fixed border-[#f0f0f0] / dark:border-slate-700 classes to the theme token-based border-border class.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A border once hardcoded in hex,
Now springs forth a token so flex!
Theme consistency gleams,
Like fresh morning dreams—
One line and the skeleton's blessed!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@Benjtalkshow
Benjtalkshow merged commit 3aa81cb into boundlessfi:main May 28, 2026
1 of 3 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