Skip to content

backport port chatbox utf8 support to 1.19.2 #758

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

Open
wants to merge 3 commits into
base: dev/1.19.2
Choose a base branch
from

Conversation

456dev
Copy link
Contributor

@456dev 456dev commented Jul 1, 2025

Backports #757 to 1.19.2

adds optional "useUtf8" parameter, informing us that the program is utf8 aware, providing its strings as utf8,
This allows sending messages in the extended minecraft character set.

see the utf8 lib in cc, as well as the \u{xxxx} escape sequence.

also sends the events with a new utf8 message parameter.

456dev added 2 commits July 1, 2025 14:03
adds optional "useUtf8" parameter, informing us that the program is utf8 aware, providing its strings as utf8,
This allows sending messages in the extended minecraft character set.

see the utf8 lib in cc, as well as the \u{xxxx} escape sequence.

also sends the events with a new utf8 message parameter.

This is a backport from IntelligenceModding#757
Copy link
Contributor

github-actions bot commented Jul 1, 2025

Build Preview

badge

You can find files attached to the below linked Workflow Run URL (Logs).

Name Link
Commit 0b3715e
Logs https://github.com/IntelligenceModding/AdvancedPeripherals/actions/runs/16004113296
Jar Files AdvancedPeripherals PR 758
Expires At 2025-09-29T18:38:26Z

@@ -349,11 +417,18 @@ public final MethodResult sendFormattedMessageToPlayer(@NotNull IArguments argum
}


// 0 message, 1 title, 2 playerName, 3 prefix, 4 brackets, 5 bracket color, 6 range, 7 utf8compatible
Copy link
Collaborator

Choose a reason for hiding this comment

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

In 0.8 we must use a keyword table argument instead of the long long argument list :(

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.

3 participants