Skip to content

test: compare Home Assistant navigation pixels - #15

Merged
efraespada merged 1 commit into
developfrom
bugfix/14-compare-navigation-screenshots-with-original-home-assistant-pixels
Nov 5, 2026
Merged

efraespada merged 1 commit into
developfrom
bugfix/14-compare-navigation-screenshots-with-original-home-assistant-pixels

Conversation

@efraespada

@efraespada efraespada commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

Text and icon navigation checks now compare every decoded screenshot pixel with the matching Home Assistant reference on macOS and Linux. This catches paint-only regressions while preserving the existing style and geometry evidence.

What changed

  • Added exact pixel-agreement checks for text and icon navigation captures on both platforms.
  • Included pixel-agreement results in each parity report alongside the existing geometry and style evidence.
  • Updated the visual parity documentation to describe the cross-platform pixel checks.

Validation

  • Local Chromium: all 16 text and 48 icon captures matched their originals exactly.
  • A 24×24 paint-only change failed both checks.
  • The 64 screenshots from the prior Linux CI artifact matched their originals exactly.
  • Type checks passed.

Closes #14

@docs-page

docs-page Bot commented Nov 5, 2026

Copy link
Copy Markdown

To preview the documentation for this pull request, visit the following URL:

docs.page/vypdev/ha-ui~15

Documentation is deployed and generated using docs.page

@vypbot vypbot added the state:ai-processing A Copilot agent is analyzing or working on the issue or change. label Nov 5, 2026
@vypbot
vypbot self-requested a review November 5, 2026 12:14
@vypbot vypbot added this to vypdev Nov 5, 2026
@vypbot vypbot moved this to In Progress in vypdev Nov 5, 2026
@vypbot
vypbot changed the base branch from develop to master November 5, 2026 12:14
@efraespada efraespada linked an issue Nov 5, 2026 that may be closed by this pull request
1 task done
@vypbot vypbot added the size: S Label to indicate a task of size S label Nov 5, 2026
@vypbot
vypbot changed the base branch from master to develop November 5, 2026 12:15
@vypbot vypbot added the 75% Progress: 75% label Nov 5, 2026
@vypbot

vypbot commented Nov 5, 2026

Copy link
Copy Markdown

Bugbot: review complete

Current status: No active findings on 46bea5a.

Pull request · Verified commit · Workflow run

@vypbot vypbot added state:ready The change is ready for human approval or merge. state:awaiting-maintainer The next action requires a maintainer response or approval. and removed state:ai-processing A Copilot agent is analyzing or working on the issue or change. labels Nov 5, 2026
@vypbot

vypbot commented Nov 5, 2026 •

Copy link
Copy Markdown

Copilot approval · Approval blocked

Status: Effective branch rules must be readable and dismiss stale approvals.
Completed: 1/1 observed checks succeeded; Bugbot complete.
Action required: Fix the named prerequisite and rerun the observer.
Impact: No new approval is confirmed for this revision.
See checks

Technical evidence

Revision: 46bea5a1a187; base: 7d5656678722; reason: unsafe-rules.

@efraespada
efraespada merged commit 202fe15 into develop Nov 5, 2026
6 of 7 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in vypdev Nov 5, 2026
@vypbot vypbot added state:verified The change has passed Copilot verification. and removed state:ready The change is ready for human approval or merge. state:awaiting-maintainer The next action requires a maintainer response or approval. labels Nov 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

75% Progress: 75% size: S Label to indicate a task of size S state:verified The change has passed Copilot verification.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

🐛🧑‍💻 - Compare navigation screenshots with original Home Assistant pixels

2 participants