Skip to content

ADFA-5442: Change the Addons Manager screen's title to match its entry point - #2051

Merged
jimturner-adfa merged 3 commits into
stagefrom
ADFA-5442-Change-window-name-to-Addons-Manager
Oct 6, 2026
Merged

jimturner-adfa merged 3 commits into
stagefrom
ADFA-5442-Change-window-name-to-Addons-Manager

Conversation

@jimturner-adfa

@jimturner-adfa jimturner-adfa commented Sep 16, 2026 •

Copy link
Copy Markdown
Collaborator

ADFA-5442

Summary

The Addons Manager screen's TopAppBar read "Plugins & Templates" - the name of its two tabs, not the feature the user actually navigated to, right above those very tabs. Since users get here by tapping "Addons Manager" in Preferences (ADFA-5640), the window title should match.

  • title_manager: "Plugins & Templates" -> "Addons Manager"
  • The "Plugins"/"Templates" tab labels themselves are unchanged - only the screen's title above them.

Testing

Verified on a physical device at font scale 1.0 and 2.0: the TopAppBar now reads "Addons Manager" cleanly, no clipping or wrapping issues at 2.0, no crashes.

🤖 Generated with Claude Code

…y point

The screen's TopAppBar read "Plugins & Templates" - the name of its
two tabs, not the feature the user actually navigated to. Since users
get here by tapping "Addons Manager" in Preferences (ADFA-5640), the
window title should say the same thing, right above those very tabs.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@coderabbitai

coderabbitai Bot commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: cc057e9f-c0b1-467d-b561-068b3f9ce762
📥 Commits

Reviewing files that changed from the base of the PR and between 4890144 and b4c4847.

📒 Files selected for processing (1)
  • resources/src/main/res/values/strings.xml
🚧 Files skipped from review as they are similar to previous changes (1)
  • resources/src/main/res/values/strings.xml

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.


📝 Summary
  • Update the Addons Manager screen title to “Addons Manager”.
  • Keep the “Plugins” and “Templates” tab labels unchanged.
  • The author reports physical-device testing at font scales 1.0 and 2.0, with no clipping, wrapping, or crashes.
  • Risk: The Preferences entry-point rename is a separate change. If this change merges first, the entry point and screen title may not match.

Walkthrough

The title_manager string resource now displays “Addons Manager” instead of “Plugins & Templates”.

Changes

Manager title

Layer / File(s) Summary
Update manager title
resources/src/main/res/values/strings.xml
The title_manager value changes to “Addons Manager”.

Priority: ⬇️ Low

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

Change: Bug fix

Suggested reviewers: davidschachteradfa

Merge Risk: ⚪ Minimal · up to b4c48

The Preferences entry and screen title match. No actionable merge-blocking issue was established.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: updating the Addons Manager screen title to match its entry point.
Description check ✅ Passed The description explains the title change, notes that the tab labels remain unchanged, and reports testing relevant to the change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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)
  • Commit to this branch
  • Create a new PR

A rabbit saw the title glow,
“Addons Manager,” neat and clear.
It twitched its nose and hopped below,
Then tucked a carrot by the screen.
“A fresh new name,” it softly sang,
And bounced away through clover green.

Comment @coderabbitai help to get the list of available commands.

@jimturner-adfa

Copy link
Copy Markdown
Collaborator Author

Merge-order note: this PR renames the screen's TopAppBar title to "Addons Manager", but the Preferences entry that navigates here ("Extensions Manager" → "Addons Manager") is a separate, not-yet-merged rename in #2048 (ADFA-5640).

If this merges before #2048, the entry point and the screen title will visibly disagree - the same mismatch this PR is meant to fix. Worth merging #2048 first, or landing both together.

@Elissa-AppDevforAll Elissa-AppDevforAll left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yep, it says Addons Manager.

@jimturner-adfa
jimturner-adfa merged commit d2af43b into stage Oct 6, 2026
5 checks passed
@jimturner-adfa
jimturner-adfa deleted the ADFA-5442-Change-window-name-to-Addons-Manager branch October 6, 2026 22:02
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.

5 participants