Skip to content

Feature - book mode - #74

Open
HoodieRocks wants to merge 29 commits into
mainfrom
feature/book-mode
Open

HoodieRocks wants to merge 29 commits into
mainfrom
feature/book-mode

Conversation

@HoodieRocks

Copy link
Copy Markdown
Contributor

This PR adds book mode, allowing books to be easily formatted

@HoodieRocks

Copy link
Copy Markdown
Contributor Author

If its not building because of package lock thats some straight BS because the git logs literally show the package lock being updated

@Silabear

Copy link
Copy Markdown
Contributor

is it good

@HoodieRocks

Copy link
Copy Markdown
Contributor Author

i havent seen any new complaints or bugs

@diamond-dev1

diamond-dev1 commented Sep 19, 2026

Copy link
Copy Markdown

Saw this comment and decided that I'm gonna have some complaints.

  • I do not like the book mode toggle button color. Why does it switch to magenta? I don't love the location of it too, but not sure where else it could be put. (edit: i made a little mockup of what would imo be a better location and design but ig just changing the magneta to gray normal would be enough)
image
  • The "book details" button should imo be smth like a pencil or book with pencil instead of a gear icon. I also feel like Generation (Original, COpy of Original, ...)

  • An option to render the book pages list smaller could be nice, as I imagine for larger books its going to be a lot of scrolling.

@HoodieRocks

Copy link
Copy Markdown
Contributor Author

Page list compact mode proposal:
image

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 19, 2026

Copy link
Copy Markdown

Deploying tools with  Cloudflare Pages  Cloudflare Pages

Latest commit: 893936b
Status: ✅  Deploy successful!
Preview URL: https://e8ad05a0.tellraw-aav.pages.dev
Branch Preview URL: https://feature-book-mode.tellraw-aav.pages.dev

View logs

@HoodieRocks

Copy link
Copy Markdown
Contributor Author

no clue why you failed there but looking at the error its just that it took too long even though nothing changed

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