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

Ports several book-related PRs from /tg/ #2454

Merged
merged 14 commits into from
Sep 8, 2024

Conversation

Absolucy
Copy link
Member

@Absolucy Absolucy commented Jun 28, 2024

About The Pull Request

Ports the following PRs from /tg/:

Mostly want this as a precursor to porting more important refactors, tbh. The book formatting stuff is just a bonus.

Changelog

🆑
add: (ChungusGamer666) Books can now be burned just like any other paper item.
add: (ChungusGamer666) You can cut pages out of books with any sharp item, not just knives or wirecutters.
fix: (ZephyrTFA) Book's no longer take your formatting and throw it out the window.
refactor: (ZephyrTFA) Book display and rendering.
add: (ChungusGamer666) You can burn bibles now! But heresy has a steep cost...
/:cl:

ChungusGamer666 and others added 5 commits June 28, 2024 03:10
For some reason, unlike other paper items, you couldn't ignite books
with a lighter or other hot item.
Now, you can!

(Also made it so the odd obscure mechanic of cutting pages out of books
can be done by any sharp item, not just wirecutters or knives)

Consistency.
~~Burning copies of WGW.~~

:cl:
add: Books can now be burned just like any other paper item.
add: You can cut pages out of books with any sharp item, not just knives
or wirecutters.
/:cl:

---------

Co-authored-by: san7890 <[email protected]>
Books didn't render markdown and instead just dumped the raw contents,
(after a html encode), into the window.
Changes them to use tgui and support markdown rendering.

Books should should look the same as the paper used to make them.
:cl:
fix: Book's no longer take your formatting and throw it out the window.
refactor: Book display and rendering
/:cl:

---------

Co-authored-by: GoldenAlpharex <[email protected]>
Co-authored-by: Mothblocks <[email protected]>
use that for book binding
## About The Pull Request

Closes tgstation/tgstation#74996
## Why It's Good For The Game
## Changelog
:cl:
fix: Bookbinder no longer takes your pen's font and throws it out the
nearest window
/:cl:
This started as a simple addition where burning a bible would curse you,
but then I realized... Bibles aren't even proper books, thus can't be
burned!
So yeah, since that is not necessary due to how atom_storage works, I
reworked that.

Because burning bibles and getting cursed for it is funny.

![image](https://github.com/tgstation/tgstation/assets/82850673/2a8489ce-ecd6-45ee-9eb9-168ff820af65)

![image](https://github.com/tgstation/tgstation/assets/82850673/ebe98ad6-2d0d-4d20-9ea1-5d472d6ca465)

:cl:
add: You can burn bibles now! But heresy has a steep cost...
/:cl:

---------

Co-authored-by: san7890 <[email protected]>
@Absolucy Absolucy merged commit 9f9f412 into Monkestation:master Sep 8, 2024
22 checks passed
@Absolucy Absolucy deleted the book-updating branch September 8, 2024 23:23
github-actions bot added a commit that referenced this pull request Sep 8, 2024
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