feat(i18n): add missing German translations and pluralization#1720
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review infoConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (5)
🚧 Files skipped from review as they are similar to previous changes (4)
📝 WalkthroughWalkthroughTranslation calls in several Vue components (MobileSidebarDrawer.vue, SidebarPanel.vue, SkipBlock.vue, diff and package pages) now pass explicit numeric counts as a third argument to i18n/$t calls to enable pluralisation; the package page link also binds a translated title and adds a screen-reader-only label for small viewports. Locale files (en, en-GB, en-US, de-DE and lunaria variants) add pluralised forms and a new compare_versions_title key. i18n/schema.json adds compare_versions_title. No exported API signatures were changed. Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
📚 Description
files_count,lines_hidden, anddeps_counttitleand small-screen-sr-onlyto the diff button on the package page, to match the other buttonsThe pluralization fixes and especially the diff button change go slightly beyond i18n. I'd be happy to split them into a separate PR if that's preferred.