Update modern-chat - Features#10473
Conversation
|
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 |
|
Ready for review when ever possible, shouldn't be any updates other than bug fixes to the features if I find any later. |
This comment was marked as spam.
This comment was marked as spam.
|
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 |
yes i'm referring to that |
Okay I've removed this feature for now |
|
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 |
Removed. Really appreciate you taking the time to review, thanks! |
Features
It would be nice to address this (or another suggestion) runelite/runelite#19861 to resolve
KeyRemappingPluginconflicts. I've solved it by duplicating the KeyRemappingPlugin functionality which I'd prefer to not do.New Tabs:
Channel Filters:
Spam Integration:
ChatFilterPlugin Support:
Other:
Add extended toggle key binding support (Mouse Button 4/5 & F13-24 support)Chat Type Filtering
Set Peek Source & Mute notifications
Tab Icons