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.

Sign up for GitHub

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

Jump to bottom

Formatting flags not allways being detected #92

Closed
rtm516 opened this issue Jul 6, 2020 · 0 comments · Fixed by #107
Closed

Formatting flags not allways being detected #92

rtm516 opened this issue Jul 6, 2020 · 0 comments · Fixed by #107
Assignees
@zml2008
Labels
area: text-serialiser/legacy type: bug
Milestone
4.0.0

Comments

@rtm516
Copy link
Member

rtm516 commented Jul 6, 2020

Description:

Using the string §r§r§c§k||§e§lProfile§c§k|| the first §k isn't picked up properly causing the resulting TextComponent to look like the below image.

Example code:

String name = "§r§r§c§k||§e§lProfile§c§k||";
TextComponent component = LegacyComponentSerializer.legacy().deserialize(name);

Debug view:

2020-07-06_23-01-08
@zml2008 zml2008 self-assigned this Jul 17, 2020
@zml2008 zml2008 added this to the 4.0.0 milestone Jul 17, 2020
zml2008 added a commit that referenced this issue Jul 18, 2020
@zml2008
Correctly handle resets followed by more styling
f1e0c02 
Fixes #92
@zml2008 zml2008 mentioned this issue Jul 18, 2020
Merged
zml2008 added a commit that referenced this issue Jul 18, 2020
@zml2008
Correctly handle resets followed by more styling
ecae453 
Fixes #92
zml2008 added a commit that referenced this issue Jul 20, 2020
@zml2008
Correctly handle resets followed by more styling
83f9e24 
Fixes #92
zml2008 added a commit that referenced this issue Jul 21, 2020
@zml2008
Correctly handle resets followed by more styling
4bf3749 
Fixes #92
Draycia pushed a commit to Draycia/adventure that referenced this issue Jul 24, 2020
@zml2008 @Draycia
Correctly handle resets followed by more styling
a4fce4a 
Fixes KyoriPowered#92
zml2008 pushed a commit that referenced this issue Dec 7, 2021
@MiniDigger
fix invalid test, add new (disabled) test, see #92
16b8993 
Signed-off-by: MiniDigger <admin@minidigger.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zml2008 zml2008

Labels
area: text-serialiser/legacy type: bug
Projects
None yet
Milestone
4.0.0
Development

Successfully merging a pull request may close this issue.

Correctly handle resets followed by more styling KyoriPowered/adventure
3 participants
@zml2008 @rtm516 @kashike