Open the Toronto survey with the city, and move the way out to the end - #83
Merged
Conversation
Thirty questions on what Toronto should do next opened on a wall of type, and the survey asks for a postal code and an email before it gives anything back. A photograph says which city's survey this is at a glance. The banner is hidden once the answers are in: the card widens from 760px to 1720px to hold the candidate comparison, and a hero at that width pushes the thing the reader came back for below the fold. The band's height and its focal point are a pair. The CN Tower spire sits about a third of the way down the frame and the waterline four-fifths, so a shorter band or a lower focus cuts one end or the other off the one landmark that says Toronto. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
"Explore the candidates" sat in the masthead, beside the survey's own title. Offered there it competed with the survey for the reader's first move: a way off the page, printed next to the reason they came to it. At the foot of the comparison it is what to do next rather than instead — the candidates it links to are the ones just named in the results. The masthead's flex classes went with it; they only existed to sit the title and the link side by side. Note that the link now appears only after submitting. Someone who lands on the survey and decides not to take it leaves by the nav. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Greptile SummaryThe PR adds a Toronto skyline banner to the unsubmitted survey and moves the candidate-discovery link from the masthead to the end of the completed comparison.
Confidence Score: 5/5The PR appears safe to merge, with the changed navigation behavior clearly documented as intentional. The banner follows the existing one-way submission state, the local image path is valid, and the relocated candidate link remains reachable in every completed-state branch.
|
| Filename | Overview |
|---|---|
| src/app/toronto/vote/2026/survey/SurveyClient.tsx | Adds the pre-submission banner and deliberately relocates the candidate link to the end of the completed results without introducing a concrete defect. |
| public/elections/toronto/2026/survey-skyline.jpg | Adds the local image asset referenced by the new Next.js Image component. |
Reviews (1): Last reviewed commit: "Move the way out of the survey to the en..." | Re-trigger Greptile
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 changes to the chrome of
/toronto/vote/2026/survey.A skyline banner above the masthead
Thirty questions on what Toronto should do next opened on a wall of type, and the survey asks for a postal code and an email before it gives anything back. A photograph says which city's survey this is at a glance.
!done.public/, like every other election photograph here, rather than hotlinked.images.unsplash.comisn't innext.config.ts'sremotePatterns, andpublic/is the only asset directory shipped into the production image. Resized to 1600×1069, 292KB.h-36 sm:h-48 md:h-64withobject-[50%_60%]keeps the full tower and the waterline at every breakpoint."Explore the candidates" moves to the end
It sat in the masthead beside the survey's own title, where it competed with the survey for the reader's first move — a way off the page, printed next to the reason they came to it. At the foot of the comparison it is what to do next rather than instead, and the candidates it links to are the ones just named in the results.
The masthead's flex classes went with it; they only existed to lay out the title and the link side by side.
Worth a reviewer's attention
photo_attributionon candidates). I don't have the photographer's name from the bare CDN URL. Happy to add a credit line if wanted.Verification
Rendered locally against a live York Factory: page returns 200, image serves, banner markup sits inside the card above the masthead, and "Explore the candidates" appears zero times on the un-submitted page.
tsc --noEmit,eslintandlint:tokensall clean.I could not screenshot in a browser (the Chrome extension isn't connected here), so I verified the crop by rendering the exact regions
object-coverwill produce at three breakpoints and inspecting those.🤖 Generated with Claude Code