Skip to content

More tweaks - #22

Merged
az0 merged 28 commits into
bleachbit:gh-pagesfrom
XhmikosR:xmr/dev
Sep 18, 2026
Merged

az0 merged 28 commits into
bleachbit:gh-pagesfrom
XhmikosR:xmr/dev

Conversation

@XhmikosR

@XhmikosR XhmikosR commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Preview: https://super-salamander-00b8ec.netlify.app

I plan to add markdownlint and html-proofer later so everything is checked.

Drop the navigation option, site subtitle and RTL styles.
Drop default and stale settings, enable strict Liquid parsing.
Use Bootstrap nav pills instead of the old nav-list styles.
Show the content before the navigation on phones.
List serve-prod in the Makefile help and ignore .jekyll-metadata.
Exit with an error code, validate the category, escape the title and split EDITOR properly.
@XhmikosR
XhmikosR marked this pull request as ready for review September 16, 2026 05:09
@XhmikosR
XhmikosR marked this pull request as draft September 16, 2026 08:41
@XhmikosR

Copy link
Copy Markdown
Contributor Author

I also have these patches: XhmikosR/docs@xmr/dev...XhmikosR:docs:xmr/dev2. Should make the style more legible. I thought I'd make a separate PR after this one is merged, but let me know if you want me to push those here, or feel free to pick them.

@XhmikosR
XhmikosR marked this pull request as ready for review September 16, 2026 15:32
@az0 az0 self-assigned this Sep 18, 2026
@az0
az0 merged commit 6188eab into bleachbit:gh-pages Sep 18, 2026
5 checks passed
@XhmikosR
XhmikosR deleted the xmr/dev branch September 18, 2026 16:29
Comment thread src/_data/faqs.yml
answer: |
Cache generally improves the performance of your computer. For example, browser cache
prevents many files (such as pieces of web pages) from being downloaded again. Getting the
file from the cache is much faster than downloading it—even with fast bandwidth.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This em-dash was here since 2018, but in 2026, it looks like AI wrote it, so okay.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I just followed the README.md instructions. Not 100% but for most things.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yeah, you caught a few violations of the explicit style guide. Thanks.

@az0

az0 commented Sep 18, 2026

Copy link
Copy Markdown
Member

Highlighting the current page looks good

image

Also, I like the mobile menu

image

I also have these patches: XhmikosR/docs@xmr/dev...XhmikosR:docs:xmr/dev2. Should make the style more legible. I thought I'd make a separate PR after this one is merged

New PR is good, thank you!

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