Skip to content

[hero-casing] Capitalize every rotating source word - #54

Merged
Smetools merged 1 commit into
mainfrom
hassan/hero-casing
Jun 14, 2026
Merged

[hero-casing] Capitalize every rotating source word#54
Smetools merged 1 commit into
mainfrom
hassan/hero-casing

Conversation

@Smetools

@Smetools Smetools commented Jun 14, 2026

Copy link
Copy Markdown
Collaborator

The homepage hero rotation mixed casing — "meeting"/"email" lowercase next to "Telegram"/"WhatsApp" capitalized. Now every word leads with a capital:

Meeting · Screen recording · Telegram · WhatsApp · Email

One array + the invisible width-ghost. Pure string change.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Style
    • Updated text capitalization in the hero section rotating list to use title case for improved visual consistency.

Meeting / Screen recording / Telegram / WhatsApp / Email — was mixed casing.
Width-ghost capitalized to match.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercel Bot commented Jun 14, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
knowcap-landing Ready Ready Preview, Comment Jun 14, 2026 11:22am

Request Review

@coderabbitai

coderabbitai Bot commented Jun 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 2870fd1d-0402-4334-8dcc-f40f4aedeb59

📥 Commits

Reviewing files that changed from the base of the PR and between d980b08 and 851a865.

📒 Files selected for processing (1)
  • app/components/home-commitment.tsx

📝 Walkthrough

Walkthrough

Two string literals in app/components/home-commitment.tsx are updated from lowercase to title case: the HERO_SOURCES array entries (Meeting, Screen recording, Email) and the cl-rot-ghost placeholder span text (Screen recording).

Changes

Hero Rotating Text Casing

Layer / File(s) Summary
Title-case rotating source text and placeholder
app/components/home-commitment.tsx
HERO_SOURCES values changed from lowercase (meeting, screen recording, email) to title case (Meeting, Screen recording, Email), and the cl-rot-ghost placeholder updated to Screen recording to match.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐇 A capital letter, a small but proud deed,
Meeting and Email now stand tall indeed.
Screen recording too joins the case,
Each word now displayed with a touch of grace.
Hops of approval from this bunny's place! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title '[hero-casing] Capitalize every rotating source word' directly and accurately describes the main change: standardizing capitalization of hero section rotating text to title case.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hassan/hero-casing

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.

@Smetools
Smetools merged commit 2f6369c into main Jun 14, 2026
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