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

feat(book): fix checkboxes #776

Merged
merged 1 commit into from
Jul 16, 2024
Merged

feat(book): fix checkboxes #776

merged 1 commit into from
Jul 16, 2024

Conversation

W95Psp
Copy link
Collaborator

@W95Psp W95Psp commented Jul 16, 2024

The book uses custom checkboxes in the quickstart chapter. When we moved the book to this repo (see
#729), we dropped the theme.

mdbook documentation suggest (see
https://rust-lang.github.io/mdBook/format/theme/index.html) copying and patching the default theme.

Since here it's such a small change, I added a small script that post-process the HTML instead.

The book uses custom checkboxes in the quickstart chapter. When we
moved the book to this repo (see
#729), we dropped the theme.

mdbook documentation suggest (see
https://rust-lang.github.io/mdBook/format/theme/index.html) copying
and patching the default theme.

Since here it's such a small change, I added a small script that
post-process the HTML instead.
Copy link
Member

@franziskuskiefer franziskuskiefer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have check boxes? Fine with me.

@W95Psp
Copy link
Collaborator Author

W95Psp commented Jul 16, 2024

Yeah, on the quick start! So that users follow the different steps one by one, as if that was a task list!

@W95Psp W95Psp added this pull request to the merge queue Jul 16, 2024
Merged via the queue into main with commit 23e01bf Jul 16, 2024
13 checks passed
@W95Psp W95Psp deleted the fix-checkbox-book branch July 16, 2024 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants