ADFA-1990: Add main screen missing tooltips#889
Conversation
📝 WalkthroughRelease NotesFeatures Added
Changes
|
| Cohort / File(s) | Summary |
|---|---|
Tooltip Tag Constants idetooltips/src/main/java/com/itsaky/androidide/idetooltips/TooltipTag.kt |
Added two new public constants: MAIN_PREFERENCES ("main.prefs") and MAIN_TERMINAL ("main.terminal") to expose tooltip identifiers. |
Main Fragment Action Integration app/src/main/java/com/itsaky/androidide/fragments/MainFragment.kt |
Added tooltip tag imports and extended action mapping logic to handle ACTION_OPEN_TERMINAL and ACTION_PREFERENCES with corresponding tooltip tag resolution and action invocations. Removed legacy onLongClick specialization for terminal action. |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
Possibly related PRs
- ADFA-2505: Computer vision tooltip #839: Adds new tooltip constants to TooltipTag.kt and wires them into action handlers for consistent tooltip integration across the codebase.
- ADFA-1383 - debugger toolbar tooltips #854: Modifies TooltipTag.kt to introduce new tooltip constants and integrates tooltip tags into UI action systems (debugger actions in that PR versus main fragment actions here).
Suggested reviewers
- Daniel-ADFA
- jomen-adfa
- itsaky-adfa
Poem
🐰 Tooltips bloom like carrots in the sun,
Terminal and preferences, now wired as one,
Tags restored with floppy-eared delight,
Actions mapped in a helpful light,
No mystery now—just clarity, hooray! ✨
🚥 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 clearly describes the main change: adding missing tooltips to the main screen, which aligns with the changeset that adds MAIN_PREFERENCES and MAIN_TERMINAL tooltip constants and their integration. |
| Description check | ✅ Passed | The description is directly related to the changeset, specifying that tooltips are being added to Preferences and Terminal actions on the main screen. |
✏️ 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.
Comment @coderabbitai help to get the list of available commands and usage tips.
Add tooltips to
PreferencesandTerminalactions on the main screen