Conversation
📝 WalkthroughRelease Notes - Git Tooltips EnhancementFeatures Added
UI Changes
Code Changes
|
| Cohort / File(s) | Summary |
|---|---|
Tooltip System idetooltips/src/main/java/com/itsaky/androidide/idetooltips/TooltipTag.kt, resources/src/main/res/values/strings.xml |
Added two new tooltip constants: MAIN_GIT ("main.git") and GIT_DOWNLOAD_SCREEN ("git.download"). Updated string resource "clone" to "download_project" with text "Download project". |
Fragment Integration app/src/main/java/com/itsaky/androidide/fragments/MainFragment.kt, app/src/main/java/com/itsaky/androidide/fragments/CloneRepositoryFragment.kt |
MainFragment now handles ACTION_CLONE_REPO by calling showCloneRepository() and returns MAIN_GIT tooltip tag. CloneRepositoryFragment attaches long-click listeners to all views (except EditText and TextInputLayout) to show tooltips via TooltipManager with GIT_DOWNLOAD_SCREEN tag. |
Layout app/src/main/res/layout/fragment_clone_repository.xml |
Updated cloneButton text from "@string/clone" to "@string/download_project". |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~12 minutes
Possibly related PRs
- PR
#1007: Implements the same pattern of adding long-press tooltip listeners to UI elements using TooltipManager and extends TooltipTag.kt constants. - PR
#1005: Introduces the clone repository feature that this PR builds upon with tooltip enhancements. - PR
#889: Extends the MainFragment's action-to-tooltip mapping system and adds new TooltipTag constants to the same centralized tooltip system.
Suggested reviewers
- itsaky-adfa
- Daniel-ADFA
- jomen-adfa
- jatezzz
Poem
🐰 A bunny hops through tooltips bright,
Long-press the views to see the light,
From main to download, labels gleam,
Fresh tags make the UI's dream,
No more mystery—just pure delight! ✨
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Docstring Coverage | Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. | Write docstrings for the functions missing them to satisfy the coverage threshold. |
✅ Passed checks (2 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | The title 'Git tooltips' clearly and concisely summarizes the main change: adding tooltip functionality to git-related UI elements (clone action and clone screen). |
| Description check | ✅ Passed | The description directly relates to the changeset by explaining that tooltips are being added to the git clone action and git clone screen, which aligns with the actual changes made. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
- 📝 Generate docstrings (stacked PR)
- 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Post copyable unit tests in a comment
- Commit unit tests in branch
ADFA-3214-git-tooltips
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 @coderabbitai help to get the list of available commands and usage tips.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@resources/src/main/res/values/strings.xml`:
- Line 1159: You renamed the string resource key from "clone" to
"download_project" which breaks localized translations; update every localized
strings XML that still uses the "clone" key (e.g., files like
termux_shared_strings.xml under values-*/ folders) to include the new key
"download_project" (either rename the key or add a duplicate entry mapping the
translated value to "download_project"), ensuring all translations that
previously targeted "clone" are present for "download_project" in this PR so
non-English locales no longer fall back to English.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: e5f6c07d-ea00-45d4-8f62-b0de6cd3efe3
📒 Files selected for processing (5)
app/src/main/java/com/itsaky/androidide/fragments/CloneRepositoryFragment.ktapp/src/main/java/com/itsaky/androidide/fragments/MainFragment.ktapp/src/main/res/layout/fragment_clone_repository.xmlidetooltips/src/main/java/com/itsaky/androidide/idetooltips/TooltipTag.ktresources/src/main/res/values/strings.xml
Add tooltips to git clone action and git clone screen
Screen_recording_20260309_225346.mp4