Repository navigation
Move weather to the left - #9
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
beejsbj
left a comment
beejsbj
left a comment
There was a problem hiding this comment.
gpt-6-sol (low effort) review of exact head 427f31c: no findings. The two-column header puts weather at the left padding edge and Menu/Done at the right, and the updated viewport checks cover both modes and control clearance. I inspected the four-file diff and nearby source; suites were run by the parent and were not rerun here.
Move weather to the left edge of the header, with Menu/Done on the right. The greeting and dial stay centered together.
Update the existing five viewport checks for left alignment in both live and shaping modes, and bump the PWA cache version to 2.2.4. All 55 unit tests and 46 browser tests pass. Visually checked at 414×896.
CI exposed an existing race in the solar marker check; it now waits for the first label render before asserting its text. The final head passes CI and all 46 browser tests.