Skip to content

feat(ADFA-3823): Rename Termux to Terminal#1330

Open
dara-abijo-adfa wants to merge 3 commits into
stagefrom
ADFA-3823-rename-termux-to-terminal
Open

feat(ADFA-3823): Rename Termux to Terminal#1330
dara-abijo-adfa wants to merge 3 commits into
stagefrom
ADFA-3823-rename-termux-to-terminal

Conversation

@dara-abijo-adfa
Copy link
Copy Markdown
Contributor

Rename "Termux" to "Terminal" on Preferences screen

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 22, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 88dcaede-6c10-43c3-b8c4-48d2758a31ed

📥 Commits

Reviewing files that changed from the base of the PR and between 4cd2b8f and 6abc033.

📒 Files selected for processing (1)
  • resources/src/main/res/values/termux_app_strings.xml

📝 Walkthrough

Release Notes - Terminal Preferences Naming Update

Changes

  • Updated preferences title: "Termux" renamed to "Terminal" in the Preferences screen (termux_preferences_title)
  • Updated preferences summary: Changed from "Preferences for the Termux terminal." to "Preferences for the terminal emulator" (termux_preferences_summary)

Affected Files

  • resources/src/main/res/values/termux_app_strings.xml

Risks & Best Practice Violations

  • ⚠️ Incomplete Localization Update: The PR updates only the base English strings file but does not update 12 localized versions (Arabic, Bengali, German, Spanish, French, Hindi, Indonesian, Portuguese, Romanian, Russian, Turkish, and Chinese). Localized files still contain the old "Termux" terminology, creating a significant inconsistency where non-English users will continue seeing outdated terminology while English users see the updated "Terminal" naming. This violates best practices for maintaining consistent user experience across all supported languages.
  • Recommendation: Localized string files in values-* directories should be synchronized with the same terminology changes to ensure all users experience the renamed feature consistently.

Walkthrough

This PR updates two preferences-related string resources in the Android strings XML: termux_preferences_title is changed to "Terminal" and termux_preferences_summary is reworded to "Preferences for the terminal emulator".

Changes

UI String Resource Update

Layer / File(s) Summary
Preferences strings text changes
resources/src/main/res/values/termux_app_strings.xml
termux_preferences_title changed from "Termux" to "Terminal"; termux_preferences_summary changed from "Preferences for the Termux terminal" to "Preferences for the terminal emulator".

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A tiny tweak, a gentle nudge,
From "Termux" to "Terminal" we trudge,
The summary hops a clearer way,
"terminal emulator" greets the day,
A rabbit cheers with a little mug ☕️

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: renaming Termux to Terminal in the preferences, which matches the changeset.
Description check ✅ Passed The description is directly related to the changeset, specifying that 'Termux' is being renamed to 'Terminal' on the Preferences screen.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ADFA-3823-rename-termux-to-terminal

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 and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

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 `@resources/src/main/res/values/termux_app_strings.xml`:
- Around line 104-105: Update the display text for the preference summary to
match the rebranded title: change the string resource termux_preferences_summary
so it no longer references "Termux terminal" and instead uses wording consistent
with termux_preferences_title (for example "Preferences for the Terminal" or
"Terminal preferences"); make this change in the same values XML entry for
termux_preferences_summary and ensure translations are handled separately from
identifiers (keep the resource name termux_preferences_title unchanged).
🪄 Autofix (Beta)

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: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9c0258d3-fa9f-498c-a16c-0442feb82bee

📥 Commits

Reviewing files that changed from the base of the PR and between 6e7add6 and 4cd2b8f.

📒 Files selected for processing (1)
  • resources/src/main/res/values/termux_app_strings.xml

Comment thread resources/src/main/res/values/termux_app_strings.xml Outdated
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.

2 participants