Skip to content

Docs: show the template's Show total switch reaching data.js - #80

Merged
SandObserver merged 1 commit into
mainfrom
docs/template-show-total
Oct 3, 2026
Merged

SandObserver merged 1 commit into
mainfrom
docs/template-show-total

Conversation

@SandObserver

Copy link
Copy Markdown
Owner

Summary

  • Sync the template widget copy from app #393: data.js and demo.js return total: null when Show total is off; the page formats the total with formatNumber and lists up to three item names.
  • Update the Build a custom widget snippets to read showTotal and format the total, and note that an unsaved toggle is absent from ctx.config.
  • Add an unused glyph and a README line to the widget checklist.

Testing
npm run build and npm run check pass. Template preview checked in headless Chromium, light and dark, against the production build: renders the total and item names with no page errors.

Checklist

  • npm run build and npm run check both pass.
  • Product UI is shown by running or capturing it, never recreated in CSS.
  • Any new preview or screenshot carries no private hostnames or real data.
  • Visual changes verified in both colour schemes and both docs themes.
  • Links to the app repo still resolve.

@SandObserver SandObserver added the documentation Improvements or additions to documentation label Oct 3, 2026
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying stackyard-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5482f4a
Status: ✅  Deploy successful!
Preview URL: https://4b55044d.stackyard-docs.pages.dev
Branch Preview URL: https://docs-template-show-total.stackyard-docs.pages.dev

View logs

@SandObserver
SandObserver merged commit 79a349c into main Oct 3, 2026
2 checks passed
@SandObserver
SandObserver deleted the docs/template-show-total branch October 3, 2026 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant