BaseUI: Clean up dead code

[Xaositek]

BaseUI: Clean up dead code

Removes:

• class DebugInfo
• void drawFrame
• void drawDebugInfoWiFiTrampoline
• drawGpsPowerStatus
• old S&F images that are unused

Summary by CodeRabbit

• Refactor
  • Removed the legacy debug display and its unused status integrations.
  • Streamlined the interface by eliminating obsolete diagnostic and GPS power-status rendering components.
  • Cleaned up unused question, information, and Store-and-Forward icon assets.
  • Simplified display-related functionality while preserving remaining UI behavior, Wi-Fi rendering, and callback handling.
  • Retained existing Wi-Fi and other screen displays.

[github-actions]

⚡ Try this PR in the Web Flasher

Note

Building this pull request… the flash button, badges and supported-board
list will appear here automatically once CI finishes.

[coderabbitai]

📝 Walkthrough

Walkthrough

The graphics cleanup removes legacy debug rendering, its declarations and dependencies, the GPS power-status renderer, and obsolete question, info, and Store-and-Forward image declarations. WiFi rendering now calls drawFrameWiFi directly.

Changes

Graphics cleanup

Layer / File(s)	Summary
Remove legacy debug rendering src/graphics/draw/DebugRenderer.cpp, src/graphics/draw/DebugRenderer.h, src/graphics/Screen.cpp	The legacy drawFrame renderer and WiFi trampoline are removed. Related dependencies and declarations are deleted. The WiFi frame callback now uses drawFrameWiFi.
Remove obsolete graphics assets and GPS API src/graphics/draw/UIRenderer.h, src/graphics/images.h	The GPS power-status declaration and obsolete question, info, and Store-and-Forward bitmap declarations are removed.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• meshtastic/firmware#11082: Removes overlapping legacy graphics and debug-rendering code.
• meshtastic/firmware#11304: Consolidates GPS rendering related to the removed GPS power-status function.

Suggested reviewers: caveman99, jp-bennett, harukitoreda

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description summarizes the code removals but omits the required testing attestations and affected-device regression results.	Add the Attestations section and state whether the changes were tested, including regression results for each affected device or explain unavailable hardware.

✅ Passed checks (4 passed)

Check name	Status	Explanation
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.
Title check	✅ Passed	The title clearly and concisely describes the removal of unused BaseUI code.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch baseui_deadcodecleanup

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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