Remove em dashes from the manifest and meta description - #19
Merged
vpetersson-bot merged 1 commit intoAug 5, 2026
Merged
Conversation
The app-store fetches this manifest at build time and renders its description straight onto signage-apps.com, so an em dash here shows up as an em dash on the store pages (/, /examples/, /world-clock/). Splits the description into two sentences and rewrites the page meta description without one. Copy only; no schema or behaviour change.
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
1 participant
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.
Why
signage-apps.com (
Screenly-Labs/app-store) fetches this app's.well-known/signage-app.jsonat build time and renders itsdescriptiondirectly onto the store pages. So an em dash in this manifest becomes an em dash on/,/examples/and/world-clock/over there, which is against the house copy style.Together with Screenly-Labs/timer#14 this clears the last em dashes on the store left over from Screenly-Labs/app-store#53; neither could be fixed from that repo, because the manifests are fetched from these origins.
What changed
Copy only, two strings:
description:…for any display — every city shows…becomes two sentences,…for any display. Every city shows…index.htmlmeta description: em dash becomes a colonNo schema, URL-parameter contract or behaviour change.
Verification
bun run lint,bun run typecheck,bun test(35 pass) andbun run buildall clean. Builtdist/.well-known/signage-app.jsonis valid JSON and both it anddist/index.htmlcontain zero em dashes.Note the store will keep serving the old text until this is deployed and the app-store rebuilds.