-
Notifications
You must be signed in to change notification settings - Fork 1
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
Changed From 100 Pages per Book to 50 #12
Conversation
also changed page line limit from 14 to 13 to prevent some words overflowing and not appearing |
I think adding a toggle to switch between a bedrock and java mode would be the better option, as that way we don't have to arbitrarily reduce the limit to 50 pages instead of 100. Regrading the cutoff of words, its been an issue for a long time, although I've tried to solve it trough many patches. Changing the line limit to 13 will indeed solve the problem, as it creates an additional fallback line in case generation is not working as intended, but we also loose 50 / 100 lines of content. However, having words disappear is arguably worse than loosing some lines, so here's what I'm thinking:
Also, thanks for bringing this to my attention, as I totally had forgot about bedrock mode as I've only ever played on java. Good catch! |
Another thing but I don't really know typescript so your code is all over
the place for me lol, but you should make a option for sending the commands
directly over rcon as generated or a secondary plugin that accepts and runs
the commands or something like that
…On Sat, Mar 16, 2024, 3:06 AM William Larsson ***@***.***> wrote:
I think adding a toggle to switch between a bedrock and java mode would be
the better option, as that way we don't have to arbitrarily reduce the
limit to 50 pages instead of 100.
Regrading the cutoff of words, its been an issue for a long time, although
I've tried to solve it trough many patches. Changing the line limit to 13
will indeed solve the problem, as it creates an additional fallback line in
case generation is not working as intended, but we also loose 50 / 100
lines of content. However, having words disappear is arguably worse than
loosing some lines, so here's what I'm thinking:
- The lines will be reduced to 13 until a proper fix for word cutoff
is implemented.
- A mode for bedrock will be created
- I will close this pull request, but open an issue regarding the
bedrock mode, and implement the 13 line limit in a separate commit ASAP
Also, thanks for bringing this to my attention, as I totally had forgot
about bedrock mode as I've only ever played on java. Good catch!
—
Reply to this email directly, view it on GitHub
<#12 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AW7UAT4RJEHIU5TKQNFU7TDYYPVO7AVCNFSM6AAAAABEV35PYGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBRHA4DGMJTGM>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
|
The default limit in Minecraft for pages in a book is 50, and bedrock users cannot read the pages after the 50th page, they just come in blank