Skip to content

Update modern-chat - Features#10473

Merged
LlemonDuck merged 4 commits intorunelite:masterfrom
BenDol:features
Apr 6, 2026
Merged

Update modern-chat - Features#10473
LlemonDuck merged 4 commits intorunelite:masterfrom
BenDol:features

Conversation

@BenDol
Copy link
Copy Markdown
Contributor

@BenDol BenDol commented Feb 5, 2026

Features

It would be nice to address this (or another suggestion) runelite/runelite#19861 to resolve KeyRemappingPlugin conflicts. I've solved it by duplicating the KeyRemappingPlugin functionality which I'd prefer to not do.


New Tabs:

  • Dedicated Game tab for game/system messages (can be disabled in the settings)
  • Dedicated Trade tab for trade messages (can be disabled in the settings)

Channel Filters:

  • Filter button on input box to toggle message types
  • Filter by: Public, Private, Clan, Friends Chat, Trade, Game, System, Auto Typed

Spam Integration:

  • Will properly account for spam filtering
  • Right-click messages to mark spam/ham
  • Shares corpus files with SpamFilterPlugin

ChatFilterPlugin Support:

  • Respects message blocking and collapse settings
  • Duplicate message collapse detection

Other:

  • Add missing Friends Chat kick option (if permitted)
  • Set peek source tab via tab right-click menu
  • Per-tab notification muting via tab right-click menu
  • Suppress peek fade at GE option (Stops the peek overlay from auto-fading in on messages while at the Grand Exchange)
  • Add extended toggle key binding support (Mouse Button 4/5 & F13-24 support)
  • Add custom Timestamp Color & Type Prefix Color configurations
  • Add ForceRecolorPlugin integration
  • Make sure the peek overlay supports message collapsing
  • Add peek source border highlight to the tab
  • Add optional report button + session timer (similar to the legacy version)
  • Add tab icon support (optional)
  • (Remove classic mode) This ends up being redundant with the new filtering features, auto close pm tab config and tab muting
  • Add a separate "Show NPC Messages" setting for peek overlay and make sure they suppress rendering rather than adding

image

Chat Type Filtering

image

Set Peek Source & Mute notifications

Image

Tab Icons

image

@runelite-github-app
Copy link
Copy Markdown

runelite-github-app Bot commented Feb 5, 2026

@BenDol BenDol mentioned this pull request Feb 5, 2026
@runelite-github-app
Copy link
Copy Markdown

runelite-github-app Bot commented Feb 5, 2026

This plugin requires a review from a Plugin Hub maintainer. The reviewer will request any additional changes if needed.


Internal use only: Reviewer details Maintainer details

@BenDol
Copy link
Copy Markdown
Contributor Author

BenDol commented Feb 12, 2026

Ready for review when ever possible, shouldn't be any updates other than bug fixes to the features if I find any later.

@BenDol BenDol reopened this Mar 16, 2026
@filipe1607

This comment was marked as spam.

@LlemonDuck
Copy link
Copy Markdown
Contributor

you should not submit arbitrary, manual KeyEvents to the canvas, it may trip anticheat measures.

you mentioned a conflict with core KeyRemapping, but that plugin doesn't do this either, it modifies the existing KeyEvent and just switches the keycode. what issue are you trying to solve?

@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label Mar 29, 2026
@BenDol
Copy link
Copy Markdown
Contributor Author

BenDol commented Mar 29, 2026

you should not submit arbitrary, manual KeyEvents to the canvas, it may trip anticheat measures.

you mentioned a conflict with core KeyRemapping, but that plugin doesn't do this either, it modifies the existing KeyEvent and just switches the keycode. what issue are you trying to solve?

Which part of the code are you referring to here? Sorry been a while since I worked on this. This is the issue with the key remapping plugin conflicts: runelite/runelite#19861 (the PR explains it and gives my ideal solution where I wouldn't need to recreate the key remapper plugin like I've done here).

Edit: If you're referring to the ExtendedKeybind feature that is a separate feature not related to the KeyRemapping issues and to be honest I can remove it if it's not viable to send key events.

@runelite-github-app runelite-github-app Bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Mar 29, 2026
@LlemonDuck
Copy link
Copy Markdown
Contributor

If you're referring to the ExtendedKeybind feature that is a separate feature

yes i'm referring to that

@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label Mar 30, 2026
@runelite-github-app runelite-github-app Bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Mar 30, 2026
@BenDol
Copy link
Copy Markdown
Contributor Author

BenDol commented Mar 30, 2026

If you're referring to the ExtendedKeybind feature that is a separate feature

yes i'm referring to that

Okay I've removed this feature for now

@LlemonDuck
Copy link
Copy Markdown
Contributor

you cannot create menu actions that will invoke send to server, even if mundane and well intended. everything else looks good (can't recommend using ahk to test your plugin tho). if you remove these two i can merge

https://github.com/BenDol/Modern-Chat/compare/263760b6a36e7bf1f93656eb40c6a711b727b696..BenDol:031396889c1b9f90bceb51c90c07b2dbbdad645e#diff-a9d0aa085026cd103a178537517e3de72ec7037f6f9bf67dd7e8c5481d81dfb4R892-R898

https://github.com/BenDol/Modern-Chat/compare/263760b6a36e7bf1f93656eb40c6a711b727b696..BenDol:031396889c1b9f90bceb51c90c07b2dbbdad645e#diff-a9d0aa085026cd103a178537517e3de72ec7037f6f9bf67dd7e8c5481d81dfb4R1525-R1531

@LlemonDuck LlemonDuck added the waiting for author waiting for the pr author to make changes or respond to questions label Apr 5, 2026
@runelite-github-app runelite-github-app Bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Apr 6, 2026
@BenDol
Copy link
Copy Markdown
Contributor Author

BenDol commented Apr 6, 2026

you cannot create menu actions that will invoke send to server, even if mundane and well intended. everything else looks good (can't recommend using ahk to test your plugin tho). if you remove these two i can merge

https://github.com/BenDol/Modern-Chat/compare/263760b6a36e7bf1f93656eb40c6a711b727b696..BenDol:031396889c1b9f90bceb51c90c07b2dbbdad645e#diff-a9d0aa085026cd103a178537517e3de72ec7037f6f9bf67dd7e8c5481d81dfb4R892-R898

https://github.com/BenDol/Modern-Chat/compare/263760b6a36e7bf1f93656eb40c6a711b727b696..BenDol:031396889c1b9f90bceb51c90c07b2dbbdad645e#diff-a9d0aa085026cd103a178537517e3de72ec7037f6f9bf67dd7e8c5481d81dfb4R1525-R1531

Removed. Really appreciate you taking the time to review, thanks!

@LlemonDuck LlemonDuck merged commit bdef366 into runelite:master Apr 6, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants