Skip to content

docs: record dashboard visual verification - #3766

Merged
lidge-jun merged 3 commits into
devfrom
codex/release-244-dashboard-evidence-07c0
Sep 6, 2026
Merged

docs: record dashboard visual verification#3766
lidge-jun merged 3 commits into
devfrom
codex/release-244-dashboard-evidence-07c0

Conversation

@lidge-jun

@lidge-jun lidge-jun commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Summary

Verification

  • GUI tree is identical to reviewed source 42689e0; its full CI34029024036 passed. Main and two independent visual reviews passed, followed by a fresh final visual gate.
  • PNG signatures and dimensions were checked, every referenced frame was inspected, and the three-scenario QA evidence checker passed. Fixture and browser ports were closed after capture.
  • No repository local suite, typecheck or build was run. This documentation-only head receives its own CI.

Desktop capture
Narrow shadow controls

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

Depends on #3764. Captures contain fixture data only; no real account or service state.

Summary by CodeRabbit

  • Documentation

    • Clarified dashboard version-label instructions: hover the version badge or value to view the full version.
    • Expanded release verification guidance, including artifact checks, validation requirements, accessibility review, and publication confirmation.
  • Verification

    • Added comprehensive dashboard visual-verification records covering responsive layouts, keyboard navigation, dark mode, reduced motion, zoom, and artifact integrity.

@coderabbitai

coderabbitai Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Team

Run ID: be7f92a5-9e08-4ec6-b498-9610aa844a43

📥 Commits

Reviewing files that changed from the base of the PR and between 381c6d8 and eb33893.

⛔ Files ignored due to path filters (12)
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-1024.png is excluded by !**/*.png
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-1440-dark.png is excluded by !**/*.png
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-1440-lower.png is excluded by !**/*.png
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-1440-zoom2.png is excluded by !**/*.png
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-1440.png is excluded by !**/*.png
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-320-empty-repeat.png is excluded by !**/*.png
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-320-interaction.png is excluded by !**/*.png
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-320-keyboard-open.png is excluded by !**/*.png
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-320-lower.png is excluded by !**/*.png
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-320.png is excluded by !**/*.png
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-390.png is excluded by !**/*.png
  • devlog/_plan/260906_release_244_followups/screenshots/dashboard-768.png is excluded by !**/*.png
📒 Files selected for processing (3)
  • devlog/_plan/260906_release_244_followups/090_dashboard.md
  • devlog/_plan/260906_release_244_followups/091_dashboard_verification.md
  • docs-site/src/content/docs/guides/web-dashboard.md

Included review availability: Your plan provides up to 10 included reviews per hour; 7 remain after this review.


📝 Walkthrough

Walkthrough

The PR updates dashboard release verification records and requirements, and changes the dashboard guide to describe the version badge or value as the hover target for the full version label.

Changes

Dashboard release documentation

Layer / File(s) Summary
Artifact verification requirements and evidence
devlog/_plan/260906_release_244_followups/090_dashboard.md, devlog/_plan/260906_release_244_followups/091_dashboard_verification.md
The release plan records artifact pinning, retention, merge-ref validation, responsive and accessibility checks, CI ownership, dependent-PR handling, and publication criteria. The verification record documents fixture-based testing, screenshots, DOM checks, layout corrections, upload constraints, and teardown results.
Version-label hover documentation
docs-site/src/content/docs/guides/web-dashboard.md:76
The guide now identifies the version badge or version value as the hover target for the full version label.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to eb338

This updates dashboard verification records and corrects the guide’s version hover target to match the existing interface. It introduces no application behavior change or remaining merge-readiness risk.

Suggested reviewers: invalid-email-address

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the primary change: documenting dashboard visual verification. It is concise, specific, and consistent with the recorded verification artifacts and documentation updates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/release-244-dashboard-evidence-07c0

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Deterministic PR hygiene checks passed.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Sep 6, 2026
Base automatically changed from codex/release-244-dashboard-07c0 to dev September 6, 2026 11:23
@lidge-jun
lidge-jun marked this pull request as ready for review September 6, 2026 11:26
@lidge-jun
lidge-jun requested a review from Ingwannu as a code owner September 6, 2026 11:26
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 6, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-06T11:30:10.621315Z eb33893 Draft marked ready
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@lidge-jun
lidge-jun merged commit a349b52 into dev Sep 6, 2026
27 of 28 checks passed
@lidge-jun
lidge-jun deleted the codex/release-244-dashboard-evidence-07c0 branch September 6, 2026 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant