Skip to content

Conversation

@Priyanshu2004-Singh
Copy link

@Priyanshu2004-Singh Priyanshu2004-Singh commented Oct 12, 2025

Large content update to customQuotes/category.json (split from UI changes).

Summary by CodeRabbit

  • New Features
    • Revamped Contributors card with richer visuals: gradient background, hover effects, and theme-aware styling for light/dark modes.
    • Increased visible contributor avatars from 7 to 8 with improved avatar presentation.
    • Replaced the action with a prominent “View All Contributors” button.
    • Added 10 new themes (e.g., catppuccin-mocha, rose-pine, ayu-light, neon-blaze, forest-path) for broader customization.
  • Improvements
    • Enhanced layout consistency with centered alignment and adjusted spacing.
    • Added runtime data validation and error handling to ensure reliable contributor loading.

Priyanshu Singh and others added 3 commits October 11, 2025 21:49
@vercel
Copy link

vercel bot commented Oct 12, 2025

Someone is attempting to deploy a commit to the shravan20's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link

coderabbitai bot commented Oct 12, 2025

Walkthrough

Refactors ContributorCard to be theme-aware with gradient styles, updated avatar display, and a new “View All Contributors” button. Adds runtime validation and error handling for contributor data. Expands themes.js with 10 new themes and minor formatting adjustments, keeping the same export symbol.

Changes

Cohort / File(s) Change summary
ContributorCard UI & logic refresh
frontend/src/components/ContributorsCard/ContributorCard.js
Introduces theme-driven styles (gradients, shadows, hover effects), increases visible avatars from 7 to 8 with new avatar styling, centers layout, replaces button text, and adds data validation/error handling for contributors.
Theme catalog expansion
src/themes/themes.js
Adds 10 new theme entries (colors for quotes/authors/backgrounds), standardizes export spacing (module.exports = themes;), no behavior change to existing themes.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User
  participant ContributorCard
  participant ThemeProvider
  participant API as Contributors API

  User->>ContributorCard: Open page
  ContributorCard->>ThemeProvider: useTheme()
  Note right of ContributorCard: Apply theme-driven styles

  ContributorCard->>API: fetch contributors
  API-->>ContributorCard: data / error

  alt Valid array
    ContributorCard->>ContributorCard: set contributors state
  else Invalid or error
    ContributorCard->>ContributorCard: log error and set empty list
  end

  ContributorCard-->>User: Render avatars (up to 8) and "View All Contributors" button
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

A nibble of gradients, a hop of sheen,
Avatars circle in a glossy sheen.
Themes sprout like clover, ten hues anew,
Data checked twice—no bugs peek through.
I thump with joy at the stylish scene—
“View All!” I squeak, with paws pristine. 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title Check ⚠️ Warning The pull request title indicates a chore updating customQuotes categories, but the actual changes are to a UI component’s theming and styling and to theme definitions in a themes.js file, making the title misleading and unrelated to the modifications. Please update the title to summarize the actual work, for example “feat: add theme-aware styling to ContributorCard and extend themes catalog,” so it clearly reflects the changes to theming and UI components.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bc57607 and f8aff6f.

📒 Files selected for processing (2)
  • frontend/src/components/ContributorsCard/ContributorCard.js (2 hunks)
  • src/themes/themes.js (1 hunks)
🧰 Additional context used
🧬 Code graph analysis (2)
src/themes/themes.js (2)
frontend/src/config/cardTemplate/index.js (2)
  • themes (1-47)
  • themes (1-47)
frontend/src/util/themes/index.js (1)
  • themes (1-242)
frontend/src/components/ContributorsCard/ContributorCard.js (2)
frontend/src/components/Pages/Home/index.js (3)
  • useStyles (10-15)
  • theme (36-36)
  • classes (75-75)
frontend/src/components/Dashboard/index.js (2)
  • theme (37-39)
  • classes (40-40)

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.

@Priyanshu2004-Singh
Copy link
Author

Updated categories.json with 150+ quote in each categroy :

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