Report HTML: add 'Fix all now' button that jumps to the CTA - #32
Merged
Conversation
Mirror of the website change in the S3-hosted report template. A primary
accent-green "Fix all now" button next to "Share this roast"; clicking it
scrolls to the end-of-report CTA ("Go back to your conversation…") and
pulses it — in-page anchor, no link.
- report-template.html: new button + FIXALL_LABEL token, id="rc-cta" on the
CTA, rc-share-primary + rc-cta-pulse CSS, scroll handler in the script.
- render-report.mjs: fills FIXALL_LABEL (一次全修 / Fix all now).
render-report.test.mjs: 17/17 pass. Rendered fixture verified in a real
browser — button green in the share row, click lands the CTA in view.
Refs #31.
Co-Authored-By: Claude Opus 4.8 <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
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.
Refs #31. Mirror of first-tree-website#205 in the S3-hosted report template.
A primary accent-green "Fix all now" button next to "Share this roast". Clicking it scrolls to the end-of-report CTA and pulses it — in-page anchor, no link.
report-template.html: new button +FIXALL_LABELtoken,id="rc-cta"on the CTA,rc-share-primary+rc-cta-pulseCSS, scroll handler in the inline script.render-report.mjs: fillsFIXALL_LABEL(一次全修 / Fix all now).render-report.test.mjs: 17/17 pass. Rendered the fixture and viewed it in a real browser — button renders green in the share row, click lands the CTA in view.🤖 Generated with Claude Code