NextCommerceCo / NextCommerceCo/spark
Suppress placeholder sections on live storefronts
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 0
- Forks
- 1
- Avg merge
- 9h 52m
- Merged PRs (30d)
- 18
Description
Problem: Empty sections render dashed-border placeholder boxes to real shoppers when a section toggle is on but content is empty. Suppressing them on live storefronts needs a platform preview or editor-mode context variable that has not been confirmed.
Evidence: partials/section_featured_products.html:17-34 and sibling section partials; TODOS.md:5-10 (open P2, blocked on the platform variable).
Done when:
- The platform confirms a template context variable that distinguishes editor/preview rendering from live rendering (or confirms none exists, which turns this into a platform request).
- Placeholders render in editor/preview only; live storefronts never show them.
- The variable is documented in the public theme docs.
Blocked on: platform confirmation of the context variable.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with partials/section_featured_products.html:17-34 and compare the sibling section partials, then read TODOS.md:5-10. First confirm with the platform whether a template context variable distinguishes editor/preview from live rendering; done means placeholders appear only in editor/preview, never on live storefronts, and the variable is documented in the public theme docs.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- documentation, frontend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100