Repository navigation
Palette 5.2.7: improve discovery, account saving and mobile interactions - #139
Merged
Merged
Conversation
Add a dated section to the 5.2.7 review checkpoint capturing the full verification run performed in the environment that imported the reviewed release/5.2.7 commit and published the PR: frontend build/lint/format, 232 vitest tests with coverage and no orphan CSS; backend ruff/mypy, 192 pytest against a locally provisioned PostgreSQL, alembic agreement and pip-audit; and 32 Playwright e2e/a11y tests. Notes which gates (visual baselines, Lighthouse, integration) remain for CI and why. This is the only change on top of the reviewed reference commit; no source was hand-edited.
Record that release/5.2.7 was pushed and PR #139 to main was opened from the publishing environment; the earlier no-credentials note no longer applies.
Ingwalde
added a commit
that referenced
this pull request
Sep 15, 2026
Record that release/5.2.7 was pushed and PR #139 to main was opened from the publishing environment; the earlier no-credentials note no longer applies. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WUEj1ssxL5BHJN6m84rL7e
The 5.2.7 card, hero, menu and layout changes altered how every visual page renders, but the reviewed release did not re-record the committed baselines, so the `visual` CI job failed on 21 screenshot diffs. Re-recorded all 21 affected baselines in the pinned Playwright image (mcr.microsoft.com/playwright:v1.62.1-noble), the same environment CI compares in, and confirmed a clean compare passes (22/22). No app or test code changed.
Ingwalde
force-pushed
the
release/5.2.7
branch
from
September 15, 2026 11:54
f3e16e2 to
a9a0cdf
Compare
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.
Palette discovery and detail actions were difficult to use on small screens, and guest
favorites obscured the requirement to save palettes to an account. This release makes
saving explicit, simplifies palette cards and groups secondary actions.
This branch imports the already-implemented and user-reviewed
release/5.2.7work 1:1 fromthe handoff bundle (reference commit
0bc2730, baseb0fc4aa= v5.2.6). The import wasverified:
git diff 0bc2730 <import>is empty and the range is 54 files, +1854/-781. Nosource was hand-edited. The only commit added on top is the review-doc update below.
Changes
Save, up to two visible tags) and shorten the editorial hero while preserving randomized
artwork (2–6 colors chosen once per home visit, not on every render).
with return-path/TTL validation; no guest local collection is restored.
pagination and total; stable tags, debounced search and URL/back-navigation + scroll state.
source=single+ handle + slug) andimprove Favorites states (account wall, empty state, Manage-gated Clear).
color-vision reflow and selection contrast, and viewport-clamped Copy/More menus with
CSS chevrons.
menus inert); respect
prefers-reduced-motion.Validation
Run in this publishing environment on the imported branch. Full detail is in
docs/releases/5.2.7-review.md.Frontend (Node 22,
npm ci):npm run build(tsc + vite): passed.npm run lint(oxlint) andnpm run format:check(prettier): passed.npx vitest run --coverage --maxWorkers=2: 232 passed / 42 files, exit 0.Coverage — statements 78.19%, branches 72.95%, functions 72.61%, lines 80.52% (thresholds met).
npm run css:orphans: no orphaned class names.Backend (venv from
backend/requirements-dev.txt, against a locally provisioned PostgreSQL 16with
pg_trgm— the Docker Composetestprofile could not run because the egress policyblocks the container image registry):
ruff check,ruff format --check,mypy backend/app: passed.pytest: 192 passed, coverage 88.92% (gate 80%), including the new regression tests forexact color-count bounds (1/8), invalid bounds and combined search/tag/count with total
counted before pagination.
alembic upgrade head→alembic check): "No new upgradeoperations detected."
pip-audit --strict: no known vulnerabilities.Playwright:
e2e/(a11y + flows + focus + email links): 32 passed, run against the environment'sChromium via a temporary, uncommitted
executablePathoverride (the pinned Playwright 1.62.1expects a browser build not installed here). No committed config or app code was changed.
Not run in this environment (left for CI): the visual screenshot baselines (only meaningful in
the pinned Playwright image with the matching browser build — baselines must not be updated
blindly), the Lighthouse budget, and the real-stack integration suite (both need container
images the egress policy blocks). No new screenshots were captured here.
The user reviewed the interactive mobile demos and approved creating this PR. No merge,
deployment or tag is included.
🤖 Generated with Claude Code
https://claude.ai/code/session_01WUEj1ssxL5BHJN6m84rL7e
Generated by Claude Code