Skip to content

tablets: Fix some issues where the tablet would not update on a server. - #82

Merged
CaitlynMainer merged 2 commits into
PC-Logix:main-MC1.21.1from
AmandaCameron:fix-tablets
Aug 26, 2026
Merged

tablets: Fix some issues where the tablet would not update on a server.#82
CaitlynMainer merged 2 commits into
PC-Logix:main-MC1.21.1from
AmandaCameron:fix-tablets

Conversation

@AmandaCameron

Copy link
Copy Markdown

No description provided.

@AmandaCameron
AmandaCameron marked this pull request as draft August 25, 2026 19:56
@AmandaCameron

Copy link
Copy Markdown
Author

This is completely incorrect to be fixing the issue. Investigating myself now.

@AmandaCameron

Copy link
Copy Markdown
Author

Force pushed with a (partial) fix. For some reason the tablet shows as on when you take it out of the charger though, but I'm not sure how to fix that, and I'm rather low on spoons for today.

@AmandaCameron
AmandaCameron marked this pull request as ready for review August 26, 2026 12:51
@AmandaCameron

Copy link
Copy Markdown
Author

Combined with 594822f (which I rearranged slightly in this PR) the tablet appears to be working as intended now.

For some reason, when taking the tablet out of a charger it will not
re-initialize the TextBuffer, causing the display to not update.
The code was unnessarially calling tablet.writeDataNBT twice, so I
rearranged the code to get rid of the duplicate call.
@CaitlynMainer
CaitlynMainer merged commit fc32b30 into PC-Logix:main-MC1.21.1 Aug 26, 2026
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.

2 participants