Skip to content

Add right-click Clear option to debug console#1682

Open
aussio wants to merge 2 commits intochsami:developmentfrom
aussio:console-clear
Open

Add right-click Clear option to debug console#1682
aussio wants to merge 2 commits intochsami:developmentfrom
aussio:console-clear

Conversation

@aussio
Copy link

@aussio aussio commented Feb 17, 2026

Summary

A small quality of life change to allow clearing the console. I like to add conditional debug logs and they get a tad bit spammy. So this is nice when troubleshooting changes.

Screen.Recording.2026-02-16.at.7.43.18.PM.mov

Test plan

  • Open the debug console via "Show console" button in the bottom of the menu.
  • Let some log output accumulate
  • Right-click the console button and select "Clear"
  • Verify the console text is cleared

chsami and others added 2 commits February 12, 2026 07:46
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@aussio aussio marked this pull request as draft February 17, 2026 01:48
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 17, 2026

Walkthrough

The changes add a console clearing feature to the RuneLite client. A "Clear" action is introduced to the console toggle button's popup menu in ClientUI.java. This action invokes a newly added clear() method in LogConsolePanel.java that clears the console text area. The clear() method uses SwingUtilities.invokeLater to ensure thread-safe execution on the Event Dispatch Thread.

Possibly related PRs

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding a right-click Clear option to the debug console, which aligns perfectly with the changeset.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main
Description check ✅ Passed The pull request description is directly related to the changeset, explaining the motivation (clearing console during debugging) and providing a clear test plan.

✏️ 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.

.tooltip("Show console")
.onClick(this::toggleConsole)
.popup(Map.of("Clear", () -> consolePanel.clear()))
.build(), false);
Copy link
Author

@aussio aussio Feb 17, 2026

Choose a reason for hiding this comment

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

I'll admit this UI a tad bit ugly. It felt like an intuitive place to put the button though.

@aussio aussio marked this pull request as ready for review February 17, 2026 01:52
@aussio aussio changed the base branch from main to development February 17, 2026 01:52
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