Skip to content

Follow source-sized absolute gallery positions across viewports - #473

Merged
chubes4 merged 1 commit into
mainfrom
fix/fluid-gallery-inset
Sep 29, 2026
Merged

chubes4 merged 1 commit into
mainfrom
fix/fluid-gallery-inset

Conversation

@chubes4

@chubes4 chubes4 commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Summary

Runtime-written absolute inset: top auto auto left coordinates remained frozen even after gallery tile widths were made fluid. Learn the source-observed vertical and horizontal offsets as separate bounded stylesheet rules. Only absolute elements with numeric four-part inset values are eligible; unrelated normal-flow top offsets retain their original behavior. Related: #436.

Evidence

At https://www.donihomeimprovement.com/kitchens/ and /work/, the previous capture matched image widths but retained x/y positions from 1440px. A fresh 17-route recapture with this change now has zero image-geometry differences on either route at 1600 and 1728; 390px interactivity still passes and the offline check reports zero findings. Other routes still fail on hero vertical offsets and Wix external requests.

A neutral browser regression proves that the original frozen 442px gallery step follows the source at 390, 768, 1440, 1600 and 1728px after script removal. Focused capture/model suites: 49 passed; TypeScript and diff checks passed.

Finalization used the operator-authorized isolated direct fallback after Homeboy controller and Lab staging failures; fresh source comparison is the end-to-end verification.

AI assistance

OpenAI GPT-6 Sol via OpenCode measured Wix gallery positions, authored the candidate and neutral regression, and compared a fresh capture with the source under Chris Huber’s direction.

@chubes4
chubes4 merged commit 3e35c82 into main Sep 29, 2026
1 check passed
@chubes4
chubes4 deleted the fix/fluid-gallery-inset branch September 29, 2026 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant