Skip to content

Redesign CodexSpeed as an evidence ledger - #4

Merged
timmyagentic merged 3 commits into
mainfrom
codex/codexspeed-production-redesign
Jul 19, 2026
Merged

Redesign CodexSpeed as an evidence ledger#4
timmyagentic merged 3 commits into
mainfrom
codex/codexspeed-production-redesign

Conversation

@timmyagentic

Copy link
Copy Markdown
Owner

Summary

  • redesign benchmark history and comparison around an evidence-ledger visual system
  • preserve run data, routing, filters, comparison selection and security behavior
  • add visual QA evidence for desktop, mobile and compare states

Verification

  • corepack pnpm check
  • 306 unit, UI and worker tests
  • 15 Playwright, accessibility, overflow and security checks
  • Wrangler dry-run and desktop/mobile browser QA passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c249fbe6e0

ℹ️ 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".

Comment thread design-qa.md Outdated

## Sources

- Selected reference: `/Users/timmy/.codex/visualizations/2026/07/19/019f79d0-42b0-7cc3-be35-4b071ec9aeec/five-site-redesign/selected/codexspeed.png`

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Remove local Codex home paths from QA doc

This new QA note commits machine-local /Users/timmy/.codex/... artifact paths, which are not usable by anyone else and violate the repository guideline in AGENTS.md to not commit “temporary Codex homes”. Replace these with repo-relative artifacts, durable links, or a summary that does not expose local Codex workspace paths.

Useful? React with 👍 / 👎.

Comment thread apps/web/src/styles.css
padding: 16px var(--page-gutter) 20px;
border-top: 2px solid var(--rule);
display: grid;
grid-template-columns: minmax(230px, 0.95fr) minmax(380px, 1.35fr) auto;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Stack the footer before tablet widths overflow

At widths just above the 840px mobile breakpoint, this desktop footer grid still requires the 230px and 380px tracks plus two 28px gaps and an unwrapped navigation column; after the footer margins and padding, an 841px viewport leaves only about 757px of content width, so the nav cannot fit and the page can gain horizontal scroll until the mobile rule starts. Move the footer stacking/wrapping breakpoint up or allow the navigation column to wrap/shrink safely.

Useful? React with 👍 / 👎.

@timmyagentic
timmyagentic merged commit e2fe26d into main Jul 19, 2026
2 checks passed
@timmyagentic
timmyagentic deleted the codex/codexspeed-production-redesign branch July 19, 2026 12:52
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