Skip to content

fix(mobile): correct the Italian draft-PR accessibility label - #6769

Merged
iscekic merged 1 commit into
mainfrom
kwf/janitor-mobile-agent-chat-570539ea4c
Sep 28, 2026
Merged

iscekic merged 1 commit into
mainfrom
kwf/janitor-mobile-agent-chat-570539ea4c

Conversation

@iscekic

@iscekic iscekic commented Sep 27, 2026

Copy link
Copy Markdown
Collaborator

Changelog for users

  • VoiceOver now announces the Italian draft-PR badge as "PR in bozza" instead of the broken "pR in bozza".

Changelog for maintainers

  • Only the Italian draft-PR accessibility label changed; the visible badge text and the other PR states are untouched.
  • The acronym now matches the "PR" casing used by the open and closed states, so check the sibling locale keys for the same defect.

E2E proof

The Italian draft-PR badge accessibilityLabel is "pR in bozza", so VoiceOver announces the broken lowercase-p acronym, while the same badge's other states use "pull request" / "PR".

Code trace: apps/mobile/src/i18n/locales/it.json:2138 changed in 8610f085a5d41766a676993c3c5855c5e2e803b8. Sense check (jev): probability 0.95

Changed lines
-      "draft": "pR in bozza",
+      "draft": "PR in bozza",
Owner request

Fix 1 janitor finding in mobile/agent-chat. Fix every one; the proof covers each.

  1. The Italian draft-PR badge accessibilityLabel is "pR in bozza", so VoiceOver announces the broken lowercase-p acronym, while the same badge's other states use "pull request" / "PR".
    Trace: apps/mobile/src/i18n/locales/it.json:2138: The Italian draft-PR badge accessibilityLabel is "pR in bozza", so VoiceOver announces the broken lowercase-p acronym, while the same badge's other states use "pull request" / "PR". (janitor area i18n, locale it).
    Files: apps/mobile/src/i18n/locales/it.json.

@iscekic iscekic added the kwf-janitor Admitted to the workflow from a janitor finding label Sep 27, 2026
@kilo-code-bot

kilo-code-bot Bot commented Sep 27, 2026 •

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The single changed line correctly fixes the Italian draft-PR accessibility label casing (pR in bozza → PR in bozza), matching the sibling PR state labels, and the locale JSON remains valid.

Files Reviewed (1 file)
  • apps/mobile/src/i18n/locales/it.json

Note: the same lowercase pR casing pattern still exists in other locale files (e.g. ca, da, es, fr, hu, mg, mk, ms, my, nb, pt, pt-BR, ro, sr, sv). Those are pre-existing unchanged lines outside this PR's scope; consider a follow-up to align them.


Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0

Review guidance: REVIEW.md from base branch main

@iscekic iscekic self-assigned this Sep 27, 2026
@iscekic iscekic added the human-ready The PR is ready for human review. label Sep 27, 2026
@iscekic iscekic added merge-by-bot and removed human-ready The PR is ready for human review. labels Sep 27, 2026
@iscekic
iscekic marked this pull request as draft September 28, 2026 08:42
@iscekic
iscekic marked this pull request as ready for review September 28, 2026 08:45
@iscekic
iscekic marked this pull request as draft September 28, 2026 08:54
@iscekic
iscekic marked this pull request as ready for review September 28, 2026 09:06
@iscekic
iscekic marked this pull request as draft September 28, 2026 09:18
@iscekic
iscekic marked this pull request as ready for review September 28, 2026 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kwf-janitor Admitted to the workflow from a janitor finding merge-by-bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants