Skip to content

ADFA-2505: Computer vision tooltip#839

Merged
dara-abijo-adfa merged 3 commits intostagefrom
ADFA-2505-computer-vision-tooltip
Jan 23, 2026
Merged

ADFA-2505: Computer vision tooltip#839
dara-abijo-adfa merged 3 commits intostagefrom
ADFA-2505-computer-vision-tooltip

Conversation

@dara-abijo-adfa
Copy link
Contributor

Set tooltip on computer vision action on the editor toolbar

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 15, 2026

📝 Walkthrough

Release Notes: Computer Vision Tooltip

  • Added tooltip support for the computer vision action in the editor toolbar

    • New tooltip tag constant EDITOR_TOOLBAR_COMPUTER_VISION = "project.layout.vision" added to idetooltips/src/main/java/com/itsaky/androidide/idetooltips/TooltipTag.kt
    • GenerateXMLAction now returns TooltipTag.EDITOR_TOOLBAR_COMPUTER_VISION from retrieveTooltipTag instead of an empty string
  • Risks / best-practice notes

    • Low risk: change is small and localized to tooltip tagging and retrieval.
    • Public constant added to a widely referenced object—naming follows existing conventions and is consistent with other editor toolbar tags.
    • No other behavioral changes detected; ensure any telemetry or tooltip display system recognizes the new tag (no code changes required in this PR).

Walkthrough

Adds a new tooltip tag constant EDITOR_TOOLBAR_COMPUTER_VISION = "project.layout.vision" and updates GenerateXMLAction.retrieveTooltipTag() to return that constant instead of an empty string.

Changes

Cohort / File(s) Summary
Tooltip Tag Constant
idetooltips/src/main/java/com/itsaky/androidide/idetooltips/TooltipTag.kt
Added public constant EDITOR_TOOLBAR_COMPUTER_VISION = "project.layout.vision"
Action Update
app/src/main/java/com/itsaky/androidide/actions/etc/GenerateXMLAction.kt
Added TooltipTag import; retrieveTooltipTag() now returns TooltipTag.EDITOR_TOOLBAR_COMPUTER_VISION instead of ""

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • jatezzz
  • Daniel-ADFA
  • itsaky-adfa
  • jomen-adfa

Poem

🐰 A tiny tag leaps into view,
A meadow code, concise and new,
Tooltips whisper, clear and bright,
XML hums with gentle light,
Hoppy changes—soft delight! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning 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 clearly summarizes the main change: adding a computer vision tooltip to the editor toolbar, which aligns with the changeset modifications.
Description check ✅ Passed The description is directly related to the changeset, explaining that it sets a tooltip on the computer vision action in the editor toolbar.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

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

Comments