Skip to content
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

feat(legacy): chatcontrol module #4836

Draft
wants to merge 7 commits into
base: legacy
Choose a base branch
from

Conversation

EclipsesDev
Copy link
Contributor

@EclipsesDev EclipsesDev commented Dec 8, 2024

TODO:

  • StackMessage Counter [valuex]
  • StackMessage remove duplicated message
  • Combined ForceUnicodeChat -> ChatControl
  • Add EN_US Translation

Closes: #4243

@EclipsesDev EclipsesDev added 🔥 enhancement New feature or request 🌕 legacy labels Dec 8, 2024
@EclipsesDev EclipsesDev added this to the b99 milestone Dec 8, 2024
@EclipsesDev EclipsesDev marked this pull request as draft December 8, 2024 09:28
@EclipsesDev EclipsesDev removed this from the b99 milestone Dec 14, 2024
@EclipsesDev
Copy link
Contributor Author

pr will be on pending

@Coccocoahelper
Copy link
Contributor

Progress?

Copy link
Contributor

@DataM0del DataM0del left a comment

Choose a reason for hiding this comment

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

  • Add a ToggleableConfigurable object like in Nextgen so the module settings aren't cluttered, and use that here for the settings for each of these.
  • chatControl in hookChatClear is declared as final, be consistent & make chatControl in hookNoLengthLimit final.

public String message;

@Inject(method = "<init>(Ljava/lang/String;)V", at = @At("RETURN"), cancellable = true)
public void injectForceUnicodeChat(String p_i45240_1_, CallbackInfo ci) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Searge be like:
❌ map the parameter names
✔️ leave it there, happens 99.99% of the time.

Comment on lines 14 to 18
val noChatClear by boolean("NoChatClear", true)
// TODO: Add stackMessage
val stackMessage by boolean("StackMessage", true)
val noLengthLimit by boolean("NoLengthLimit", true)
val forceUnicodeChat by boolean("ForceUnicodeChat", false)
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a ToggleableConfigurable object like in Nextgen so the module settings aren't cluttered, and use that here for the settings for each of these.

@opZywl
Copy link
Contributor

opZywl commented Jan 27, 2025

add second chat pleas

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔥 enhancement New feature or request 🌕 legacy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants