Skip to content

New Crowdin updates - #370

Merged
samolego merged 2 commits into
masterfrom
l10n_master
Aug 8, 2026
Merged

New Crowdin updates#370
samolego merged 2 commits into
masterfrom
l10n_master

Conversation

@samolego

@samolego samolego commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Bug Fixes
    • Corrected Hungarian translations in uninstall and reinstall success messages.
    • Improved select-all guidance wording, punctuation, and quotation marks.
    • Removed unnecessary trailing whitespace from localized text.
  • New Features
    • Added Hungarian translations for advanced settings, donation, authentication, and USB debugging options.

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The PR corrects Hungarian spelling and formatting, and adds Hungarian translations for advanced settings, donation, authentication, and USB debugging messages.

Changes

Hungarian localization updates

Layer / File(s) Summary
Correct Hungarian resource strings
app/src/main/res/values-hu-rHU/strings.xml
Corrects uninstall and reinstall success messages. Updates select-all punctuation, quotation marks, and trailing whitespace.
Add Hungarian feature translations
app/src/main/res/values-hu-rHU/strings.xml
Adds Hungarian translations for advanced settings, donation, authentication, and USB debugging strings.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • samolego/Canta#290: Introduces Hungarian localization entries updated by this PR.
  • samolego/Canta#295: Modifies the same Hungarian resource file and localization keys.
  • samolego/Canta#357: Updates overlapping localized strings, including USB debugging and select-all messages.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately identifies the changes as new Crowdin localization updates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch l10n_master

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 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.

Inline comments:
In `@app/src/main/res/values-hu-rHU/strings.xml`:
- Around line 54-55: Update the select_all_tip and select_all_enabled string
values to use balanced Hungarian quotation marks around the quoted labels, and
correct the article before “Mind kijelölése” to match Hungarian grammar.
Preserve the existing message meanings and formatting placeholders.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b8eb0161-03f1-415a-912d-782588b506ac

📥 Commits

Reviewing files that changed from the base of the PR and between 2576eb9 and 07239a4.

📒 Files selected for processing (1)
  • app/src/main/res/values-hu-rHU/strings.xml

Comment thread app/src/main/res/values-hu-rHU/strings.xml

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 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.

Inline comments:
In `@app/src/main/res/values-hu-rHU/strings.xml`:
- Line 115: Update the Hungarian enable_usb_debugging string to use the standard
compound spelling “USB-hibakeresés” in the user-facing label.
- Around line 101-105: Translate the English values for the listed string
resources in the Hungarian resource file: click_to_expand, bloat_list_url,
bloat_list_url_description, allow_unsafe_selections,
allow_unsafe_uninstalls_description, success, hide_success_dialog,
hide_success_dialog_description, and require_auth_setting_desc. Preserve each
resource name and XML structure while replacing only the fallback text with
Hungarian translations.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 36b46bec-2ccd-4f0e-aa2d-3ed5310e4cbf

📥 Commits

Reviewing files that changed from the base of the PR and between 07239a4 and 05a6362.

📒 Files selected for processing (1)
  • app/src/main/res/values-hu-rHU/strings.xml

Comment on lines 101 to 105
<string name="click_to_expand">Click to expand</string>
<string name="bloat_list_url">Bloat list URL</string>
<string name="bloat_list_url_description">Where to get badges and descriptions from.</string>
<string name="allow_unsafe_selections">Allow unsafe selections</string>
<string name="allow_unsafe_uninstalls_description">Whether to allow selecting apps marked as unsafe (applying presets bypasses this already).</string>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "Files:"
fd 'strings\.xml$' . | sort

echo
echo "Hungarian target file size and relevant lines:"
wc -l app/src/main/res/values-hu-rHU/strings.xml
nl -ba app/src/main/res/values-hu-rHU/strings.xml | sed -n '1,180p'

echo
echo "Search target resource name usage:"
rg -n 'name="(click_to_expand|bloat_list_url|bloat_list_url_description|allow_unsafe_selections|allow_unsafe_uninstalls_description|success|hide_success_dialog|hide_success_dialog_description|require_auth_setting_desc)"' app/src/main/res || true

echo
echo "Search target resource name implementation/usages:"
rg -n '`@string/`(click_to_expand|bloat_list_url|bloat_list_url_description|allow_unsafe_selections|allow_unsafe_uninstalls_description|success|hide_success_dialog|hide_success_dialog_description|require_auth_setting_desc)|click_to_expand|bloat_list_url|bloat_list_url_description|allow_unsafe_selections|allow_unsafe_uninstalls_description|success|hide_success_dialog|hide_success_dialog_description|require_auth_setting_desc' app/src/main/res/app/src/main/java app/src/main/kotlin 2>/dev/null || true

Repository: samolego/Canta

Length of output: 2259


🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "Hungarian target file contents:"
awk '{printf "%5d\t%s\n", NR, $0}' app/src/main/res/values-hu-rHU/strings.xml

echo
echo "Target string values in Hungarian and default resources:"
python3 - <<'PY'
from pathlib import Path
import re
files = {
    "hungarian": Path("app/src/main/res/values-hu-rHU/strings.xml"),
    "default": Path("app/src/main/res/values/strings.xml"),
}
names = [
    "click_to_expand",
    "bloat_list_url",
    "bloat_list_url_description",
    "allow_unsafe_selections",
    "allow_unsafe_uninstalls_description",
    "success",
    "hide_success_dialog",
    "hide_success_dialog_description",
    "require_auth_setting_desc",
]
pattern = re.compile(r'<string name="([^"]+)"[^>]*>(.*?)</string>', re.S)
for lang, path in files.items():
    print(f"--- {lang}: {path} ---")
    text = path.read_text()
    entries = {m.group(1): " ".join(m.group(2).split()) for m in pattern.finditer(text)}
    for name in names:
        value = entries.get(name)
        print(f"{name}: {value!s}")
PY

echo
echo "Usages of target string references/usages:"
rg -n '`@string/`(click_to_expand|bloat_list_url|bloat_list_url_description|allow_unsafe_selections|allow_unsafe_uninstalls_description|success|hide_success_dialog|hide_success_dialog_description|require_auth_setting_desc)' app || true

Repository: samolego/Canta

Length of output: 11098


Translate the remaining English Hungarian fallbacks.

The Hungarian file still falls back to English for: click_to_expand, bloat_list_url, bloat_list_url_description, allow_unsafe_selections, allow_unsafe_uninstalls_description, success, hide_success_dialog, hide_success_dialog_description, and require_auth_setting_desc. Translate these values to keep the settings and auth/error screens fully localized.

🤖 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 `@app/src/main/res/values-hu-rHU/strings.xml` around lines 101 - 105, Translate
the English values for the listed string resources in the Hungarian resource
file: click_to_expand, bloat_list_url, bloat_list_url_description,
allow_unsafe_selections, allow_unsafe_uninstalls_description, success,
hide_success_dialog, hide_success_dialog_description, and
require_auth_setting_desc. Preserve each resource name and XML structure while
replacing only the fallback text with Hungarian translations.

<string name="require_auth_setting">Require authentication</string>
<string name="enable_usb_debugging">(Optional) Enable USB debugging</string>
<string name="require_auth_setting">Azonosítás szükséges</string>
<string name="enable_usb_debugging">(Opcionális) Az USB hibakeresés engedélyezése</string>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🌐 Web query:

Magyar helyesírás USB hibakeresés hivatásos nyelvvizsga USB-hibakeresés

💡 Result:

Az „USB-hibakeresés” kifejezés helyesírása az AkH. (A magyar helyesírás szabályai) 12. kiadása szerinti szabályok alapján az alábbiak szerint alakul: Az USB-hibakeresés írásmódja kötőjeles, mivel egy mozaikszóból (USB) és egy összetett főnévből (hibakeresés) álló összetételről van szó. A mozaikszóhoz járuló utótagot az AkH. 289. a) pontja értelmében kötőjellel kapcsoljuk. Bár a technikai szakirodalomban és a felhasználói felületeken (pl. Android menüpontok) gyakran találkozhatunk a kötőjel nélküli „USB hibakeresés” formával [1][2][3], a helyesírási szabályok alapján a kötőjeles írásmód a javasolt [4][5][6]. Összegzés: - Helyes írásmód: USB-hibakeresés [4][5][6] - Gyakran előforduló (de helyesírási szempontból kevésbé pontos) forma: USB hibakeresés [1][2][3] A kifejezés az Android operációs rendszer egy speciális fejlesztői beállítása, amely lehetővé teszi az eszköz és a számítógép közötti mélyebb szintű, ADB (Android Debug Bridge) protokollon alapuló kommunikációt [2][3][7]. Biztonsági okokból ajánlott csak akkor engedélyezni, ha konkrét fejlesztői, tesztelési vagy rendszer-helyreállítási feladatra van szükség [2][8][7].

Citations:


Use USB-hibakeresés in the Hungarian label.

The label contains a user-facing string, so apply the standard compound spelling.

Proposed fix
-    <string name="enable_usb_debugging">(Opcionális) Az USB hibakeresés engedélyezése</string>
+    <string name="enable_usb_debugging">(Opcionális) Az USB-hibakeresés engedélyezése</string>
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<string name="enable_usb_debugging">(Opcionális) Az USB hibakeresés engedélyezése</string>
<string name="enable_usb_debugging">(Opcionális) Az USB-hibakeresés engedélyezése</string>
🤖 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 `@app/src/main/res/values-hu-rHU/strings.xml` at line 115, Update the Hungarian
enable_usb_debugging string to use the standard compound spelling
“USB-hibakeresés” in the user-facing label.

@samolego
samolego merged commit d915cfa into master Aug 8, 2026
1 check passed
@samolego
samolego deleted the l10n_master branch August 8, 2026 09:07
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.

1 participant