Repository navigation
Fix the favicon, and reword the copy that leaned on em dashes - #45
Merged
Merged
Conversation
The link tag declared sizes="48x48", carried over from the old site. The .ico actually holds 16, 24, 32 and 48px entries, so claiming a single size stopped browsers picking the right one for a tab. It is now sizes="any". Adds 16 and 32px PNGs generated from the 512px brand icon, which browsers prefer where offered and which are sharper than the .ico's BMP-encoded entries. 852 and 2281 bytes. The .ico stays as the fallback. Both are asserted in the build, so a future change cannot drop them silently. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Six em dashes across the copy this site added, each rewritten rather than
swapped for another mark, since an em dash usually hides a sentence that
wanted restructuring:
"...to do the actual talking — there's no way around it"
-> "...to do the actual talking, and there is no way around it"
"It does not need to be fast — it only relays messages"
-> "It only relays messages, so it does not need to be fast"
"Managed Macs usually block both — and doing it anyway..."
-> "Managed Macs usually block both, and doing it anyway..."
"Yes — this is built for you"
-> "Yes, this is built for you"
"nothing sent anywhere — it all runs in your browser"
-> "questions, answered in your browser. No email address, and nothing
is sent anywhere."
"Copy failed — press Control or Command + C"
-> "Copy failed. Press Control or Command + C instead."
The FAQ, install guide and legal documents are untouched: that copy is
carried over verbatim and is not ours to restyle. None contained an em
dash in any case.
The built output now contains none at all.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two unrelated bits of polish, one commit each.
Favicon
The link tag declared
sizes="48x48", carried over from the old site. The.icoactually contains 16, 24, 32 and 48px entries, so claiming a single size stopped browsers picking the right one for a tab.sizes="any"on the.ico, so the browser chooses from what's really in there.ico's BMP-encoded entries.icostays as the fallbackThe file itself was fine and served with the right content-type, so this is purely the markup plus better sources.
Em dashes
Six across the copy this site added, each rewritten rather than swapped for another mark — an em dash usually hides a sentence that wanted restructuring:
The FAQ, install guide and legal documents are untouched — that copy is carried over verbatim and isn't ours to restyle. As it happens none of them contained one.
The built output now contains zero em dashes.
🤖 Generated with Claude Code