Repository navigation
Restore browser verification after the homepage design review - #94
Closed
openbotgit[bot] wants to merge 1 commit into
Closed
openbotgit[bot] wants to merge 1 commit into
openbotgit[bot] wants to merge 1 commit into
Conversation
luinbytes
marked this pull request as ready for review
November 12, 2026 00:35
9 of 11 tasks
Contributor
Author
|
Superseded by merged #93, now on master at This PR's
No unique needed fix remains here. Current master's verification, build and deployment all passed. Leaving this PR open for Lu; no duplicate implementation is needed. |
Owner
|
Closing this one. All of its fixes landed through #93, and QA has verified the live site (no feed button, open-water clicks feed in both modes, navigation clicks do not feed). Nothing here needs merging. |
6 of 9 tasks
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.
Description
The browser suite still expected the tabs, About label and feeding gestures removed by #92. Update those checks for the four project cards, More navigation, selectable copy, single click feeding and descriptive share card alt. Preserve the viewport matrix, food reservations, both pond modes, mobile touch rejection, reduced motion and security coverage. No tests are skipped.
Also fix an actual #92 regression: the embedded Instrument Serif font violated
font-src 'self'. Bundle the identical font bytes as a local asset and verify the font loads. The security policy and design remain unchanged. No #93 feeding access changes are included.Base:
d3bb71254ae1e6de4c0a0c248827cbee5812b284.Verified head:
313659fad05add7ef4e2450017da0f4e9d0e08ae.Failure map and local evidence covers every failure from the original Actions run, the reproduced CSP error and the font hash comparison.
Type of Change
Testing
npm run lintpassesnpm run buildsucceeds throughnpm run test:e2enpm run test:e2e: all 49 tests pass in 214.724 seconds, no skips. Original failing coverage passes in this production export rerun.npm run test:pond: 54 tests pass.npm audit --audit-level=highandgit diff --checkpass. Seven existing moderate audit findings remain; dependencies are unchanged.Screenshots (if applicable)
Local generated evidence:
test-results/project-grid-{320,390,1440}.pngandtest-results/portfolio-final-{desktop,mobile}.png. Reviewed font readability and horizontal containment. Full page screenshots contain fixed element and offscreen animation artifacts; viewport assertions are the layout evidence.