-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Compact Messages #381
Add Compact Messages #381
Conversation
src/main/java/mod/acgaming/universaltweaks/tweaks/misc/chat/TextComponentCount.java
Outdated
Show resolved
Hide resolved
@Shadow public abstract int getChatWidth(); | ||
|
||
@Inject(method = "printChatMessageWithOptionalDeletion", at = @At("HEAD")) | ||
public void utCompactMessage(ITextComponent chatComponent, int chatLineId, CallbackInfo ci) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this injection is targeted to printChatMessageWithOptionalDeletion
, please check if this still works after setting chat opacity and whatnot in the settings and coming back to it, since that calls setChatLine
directly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this injection is targeted to printChatMessageWithOptionalDeletion, please check if this still works after setting chat opacity and whatnot in the settings and coming back to it, since that calls setChatLine directly.
I've found some issues regarding the chat options and fixed them correspondingly and changed the Injection target to setChatLine
directly instead as well to prevent unforeseen issues from happening by doing not so.
Feel free to tell me when this is ready to merge in your opinion! |
This PR adds an option to remove duplicated messages in the chat and instead display a number behind it, how often the message was repeated.