Add Polish translations#734
Conversation
|
|
📝 WalkthroughWalkthroughThis PR adds Polish ( ChangesPolish locale addition
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested labels: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
|
There was a problem hiding this comment.
🧹 Nitpick comments (1)
src/components/video-editor/SettingsPanel.tsx (1)
916-916: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winConsider centralizing locale display labels to avoid sync drift.
Both
LOCALE_LABELSinMorePopover.tsxandAPP_LANGUAGE_LABELSinSettingsPanel.tsxnow duplicate the samepl: "Polski"entry. Each new locale requires updating both maps manually. Extracting a single sharedLOCALE_LABELS: Record<AppLocale, string>intosrc/i18n/config.tsand importing it in both components would eliminate this duplication and prevent future drift.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/components/video-editor/SettingsPanel.tsx` at line 916, The locale label maps are duplicated between SettingsPanel and MorePopover, including the new pl label, which can drift over time. Move the shared locale display labels into a single LOCALE_LABELS constant in src/i18n/config.ts (typed as Record<AppLocale, string>) and update SettingsPanel and MorePopover to import and use that shared source instead of maintaining separate copies.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@src/components/video-editor/SettingsPanel.tsx`:
- Line 916: The locale label maps are duplicated between SettingsPanel and
MorePopover, including the new pl label, which can drift over time. Move the
shared locale display labels into a single LOCALE_LABELS constant in
src/i18n/config.ts (typed as Record<AppLocale, string>) and update SettingsPanel
and MorePopover to import and use that shared source instead of maintaining
separate copies.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: e641023b-a36c-4374-97fe-b4cd412c5358
📒 Files selected for processing (12)
src/components/launch/popovers/MorePopover.tsxsrc/components/video-editor/SettingsPanel.tsxsrc/contexts/I18nContext.tsxsrc/i18n/config.tssrc/i18n/locales/pl/common.jsonsrc/i18n/locales/pl/dialogs.jsonsrc/i18n/locales/pl/editor.jsonsrc/i18n/locales/pl/extensions.jsonsrc/i18n/locales/pl/launch.jsonsrc/i18n/locales/pl/settings.jsonsrc/i18n/locales/pl/shortcuts.jsonsrc/i18n/locales/pl/timeline.json
|
This is a manual contribution adding Polish localization support to Recordly. What I've changed changed:
How did i perform validation:
|
Description
Adds full Polish (
pl) localization support to Recordly.This PR registers
plas a supported app locale, adds complete Polish translations for all current i18n namespaces, and exposes Polish in the language selectors used in the launch UI and editor settings.Motivation
Recordly already supports multiple UI languages, but Polish was not available.
This change adds Polish as a first-class locale so Polish-speaking users can use the launch UI, editor, dialogs, settings, shortcuts, timeline labels, and extensions UI in their language.
Type of Change
Related Issue(s)
N/A
Screenshots / Video
Screenshot (if applicable):


Video (wherever possible):
N/A
Testing Guide
Note: npm run i18n:check currently reports pre-existing failures in other locales unrelated to this PR.
Checklist
Summary by CodeRabbit