Skip to content

fix: preserve locale for CTA navigation#398

Merged
ViktorSvertoka merged 1 commit into
developfrom
fix/locale-navigation
Mar 9, 2026
Merged

fix: preserve locale for CTA navigation#398
ViktorSvertoka merged 1 commit into
developfrom
fix/locale-navigation

Conversation

@TiZorii
Copy link
Copy Markdown
Collaborator

@TiZorii TiZorii commented Mar 9, 2026

Summary by CodeRabbit

  • Style

    • Updated gradient styling across hero sections, pricing cards, and feature sections with new color token variants
    • Refined blur effects and backdrop dimensions for enhanced visual appearance
    • Adjusted responsive width constraints on various UI components for improved mobile layout
    • Narrowed maximum input widths for better form design
  • Improvements

    • Enhanced routing implementation for more consistent navigation behavior across the application

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Mar 9, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
devlovers-net Ignored Ignored Mar 9, 2026 8:17pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a4920732-a62d-4e33-a3c2-8e055764b30f

📥 Commits

Reviewing files that changed from the base of the PR and between ae83dc2 and 83a6a7f.

📒 Files selected for processing (7)
  • frontend/app/[locale]/admin/shop/products/_components/ProductForm.tsx
  • frontend/components/about/PricingSection.tsx
  • frontend/components/about/TopicsSection.tsx
  • frontend/components/admin/quiz/CreateQuizForm.tsx
  • frontend/components/dashboard/QuizResultRow.tsx
  • frontend/components/home/FeaturesHeroSection.tsx
  • frontend/components/shop/ProductFilters.tsx

📝 Walkthrough

Walkthrough

This PR consolidates routing imports across seven components by redirecting them from Next.js standard modules (next/navigation, next/link) to a local i18n routing module (@/i18n/routing). Additionally, it updates styling by modernizing gradient utility classes and adjusting responsive width constraints.

Changes

Cohort / File(s) Summary
Router Hook Migration
frontend/app/[locale]/admin/shop/products/_components/ProductForm.tsx, frontend/components/admin/quiz/CreateQuizForm.tsx, frontend/components/shop/ProductFilters.tsx
Migrates useRouter and usePathname imports from next/navigation to @/i18n/routing for centralized i18n-aware routing. No behavioral changes to navigation logic.
Link & Gradient Styling Updates
frontend/components/about/PricingSection.tsx, frontend/components/about/TopicsSection.tsx, frontend/components/home/FeaturesHeroSection.tsx
Replaces Link import with i18n routing variant and updates gradient classes from bg-gradient-to-* to bg-linear-to-* syntax with adjusted color tokens. Also modernizes blur effects (e.g., blur-[40px]blur-2xl).
Responsive Layout & Navigation Refinement
frontend/components/dashboard/QuizResultRow.tsx
Migrates router import, updates quiz-review navigation URL (removes locale prefix), and adjusts responsive width constraints for mobile display (e.g., max-w-[70px]max-w-17.5).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • DevLoversTeam/devlovers.net#65: Implements locale-preserving navigation and language-switch logic within the @/i18n/routing module that these components now depend upon.

Suggested reviewers

  • AM1007
  • ViktorSvertoka

Poem

🐰 Hops skip through next/navigation's door,
To @/i18n's locale-aware core,
Gradients shimmer in linear light,
Responsive widths fit just right,
One hop brings routing home to roost!

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: preserve locale for CTA navigation' accurately reflects the core objective of this PR, which involves replacing Next.js navigation imports with localized routing to preserve locale information across navigation actions.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/locale-navigation

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

@ViktorSvertoka ViktorSvertoka merged commit e1cad01 into develop Mar 9, 2026
8 checks passed
@ViktorSvertoka ViktorSvertoka deleted the fix/locale-navigation branch March 9, 2026 20:27
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.

2 participants