Skip to content

Conversation

@EForce11
Copy link

@EForce11 EForce11 commented Jan 28, 2026

his PR fixes hardcoded English strings in index.hbs, navigation.hbs, and other partials by wrapping them in the {{t}} helper. This enables full localization support for the theme.

Changes:

  • Wrapped "Latest" in index.hbs
  • Wrapped "Sign in" / "Subscribe" in navigation
  • Added missing helpers to placeholders.

Note

Replaces hardcoded English UI text with {{t}} to enable localization across key templates.

  • Navigation: localized burger aria-label, Sign in, Subscribe, Account, and search toggle aria-label
  • Feeds and lists: Featured, Latest, See all, sidebar Subscribe/Upgrade, and Recommendations
  • Search and header: search placeholder button Search posts, tags and authors
  • Footer: Powered by
  • Post UI: By in post cards and Read more related section
  • Lightbox controls: button titles (Close (Esc), Share, Toggle fullscreen, Zoom in/out, Previous, Next)
  • Email signup: submit button label and aria-label

Written by Cursor Bugbot for commit 48f1757. This will update automatically on new commits. Configure here.

@coderabbitai
Copy link

coderabbitai bot commented Jan 28, 2026

Walkthrough

This PR localizes user-facing text across 10 Handlebars templates by replacing hard-coded strings with the translation helper (e.g., {{t "..."}}). Changes include headings, button labels, aria-labels, and other UI text in partials such as featured, footer, header-content, navigation, post-list, email-subscription, lightbox, post-card, search-toggle, and post. No control flow, structural, or behavioral changes were made.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: wrapping hardcoded strings in i18n translation helpers across multiple template files.
Description check ✅ Passed The description is related to the changeset, detailing which UI strings were wrapped with translation helpers across multiple template files to enable localization.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

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