tweak(locales): shorten long UI labels - #5403
Conversation
Trim wordy warnings, tab labels and button labels so they fit better on mobile and leave more room for longer translations. Tab labels reuse the existing short keys instead of shortening strings shared with other screens.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Team Run ID: ⛔ Files ignored due to path filters (4)
📒 Files selected for processing (2)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. WalkthroughThe change updates translation keys for publisher status tabs and meeting schedule tabs. No logic, data flow, or control flow changed. ChangesTab label translation updates
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change shortens publisher-status and meeting-schedule tab labels for mobile layouts without altering tab behavior. No current merge-readiness risk remains. 🚥 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 |
ux-git
changed the title
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
|
Tick the box to add this pull request to the merge queue (same as
|
|
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
1 participant
Description
Several UI labels were long enough to wrap or shrink on mobile, and translations of them run considerably longer than the English source. This shortens the wordiest warnings, buttons and tab labels without losing any context, since the surrounding screen already provides it.
Only the English source strings are edited — no keys are renamed and no other locale files are touched, so existing translations stay attached and Crowdin just marks them for review.
Warnings
tr_personAlreadyAssignmentWeektr_repeatedMonthlyWarningDesctr_notFoundReviewAssignmentQualificationstr_personHasNoAssignmentHistorytr_repeatedMonthlyWarningDescis shared by the person selector and the helper line under the "Show repeated assignment warning – monthly" toggle in weekend meeting settings. The old sentence started with "This person", which never fitted the settings screen. The new wording works in both places and now reads as a preview of the warning the toggle enables, so the string stays unified.Buttons
tr_manageUserProfileSettingstr_assignmentsDeleteMultipletr_addCustomMeetingParttr_reactivatePublisherTabs
The weekly schedules tabs now read "Midweek" and "Weekend", and the publisher records tabs "Active" and "Inactive". The strings themselves are not shortened, because
tr_midweekMeetingis used in 15 other places (person page headers, dashboard links, attendance records, the S-88 export) andtr_activePublishersin 6, including the S-21 export file names. Instead the two tab call sites now use the short keys that already exist in the codebase:tr_midweek/tr_weekendandtr_active/tr_inactive. Everything else keeps the full wording.Type of change
Checklist: