chore(brand): replace homepage hero artwork#55
Conversation
Deploying hawkinsoperations-website with
|
| Latest commit: |
88b7d0d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://500d8fc0.hawkinsoperations-website.pages.dev |
| Branch Preview URL: | https://chore-homepage-hero-artwork.hawkinsoperations-website.pages.dev |
There was a problem hiding this comment.
💡 Codex Review
https://github.com/HawkinsOperations/hawkinsoperations-website/blob/88b7d0dbf55aa6d316eeed0c8b4a024f4fde8d9b/public/brand/hawkinsoperations-hero-mobile.png#L1
Restore portrait mobile hero artwork
On viewports <=720px, OrbitalHawkHero selects hawkinsoperations-hero-mobile.png, and the mobile CSS gives .ohh__brand-img a tall fixed height with object-fit: contain; this changed asset is now the same 2508×627 landscape PNG as the wide hero, whereas the previous mobile file was 941×1672 portrait. On phones this will render the hero art as a short strip with large empty/letterboxed space instead of filling the mobile frame, so the mobile-specific source should remain portrait or the mobile layout needs to be adjusted with it.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Reviewed PR #55 for scope and proof boundary. Confirmed this is a visual-only homepage hero artwork replacement affecting only:
No homepage copy, navigation, routes, metrics, detection language, runtime claims, proof claims, or public discovery metadata are included. Validation reported passing:
Approved for visual-only merge. |
Summary:
Files changed:
Validation:
Proof boundary:
Reviewer note: