Skip to content

Update expected UI screenshots for the new headless Chrome (Node 24 / Puppeteer 24) - #114

Merged
sgiehl merged 4 commits into
prepare6xfrom
dev-20341
Jul 24, 2026
Merged

Update expected UI screenshots for the new headless Chrome (Node 24 / Puppeteer 24)#114
sgiehl merged 4 commits into
prepare6xfrom
dev-20341

Conversation

@sgiehl

@sgiehl sgiehl commented Jul 21, 2026

Copy link
Copy Markdown
Member

Updates the expected UI test screenshots for the new headless Chrome used by the modernized screenshot-testing harness (Node 24 / Puppeteer 24).

This is the submodule counterpart of the core migration PR and must be merged together with it:
matomo-org/matomo#24677

The screenshot changes are rendering drift from the new headless Chrome (identical content, minor line-height/anti-aliasing shifts); each was reviewed against the previous expected before syncing.

Checklist

  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules

sgiehl added 3 commits July 21, 2026 08:31
Rendering drift only (identical content, minor line-height/AA shifts) under the
new headless Chrome used by the migrated screenshot-testing harness.
Match the Node 24 / Puppeteer 24 screenshot-testing stack; the UI job was still pinned to Node 16.
@sgiehl
sgiehl marked this pull request as ready for review July 24, 2026 08:52
@sgiehl
sgiehl merged commit a7a2733 into prepare6x Jul 24, 2026
12 checks passed
sgiehl added a commit that referenced this pull request Aug 10, 2026
* Prepare for Matomo 6

Bump plugin version to 6.0.0 and require Matomo >=6.0.0-b1,<7.0.0-b1.

* Run tests on same PHP versions as core (8.1, 8.5)

* Run plugin tests against MySQL 8.0 and MariaDB 10.6 (#113)

* Run plugin tests against MySQL 8.0 and MariaDB 10.6

The workflow inherited the shared action's MySQL 5.7 default, which is below
Matomo's new minimum. Pin the database-backed jobs to the supported floor:
PluginTests now runs a MySQL 8.0 + MariaDB 10.6 matrix, and the UI/JS jobs run
on MySQL 8.0.

* Only upload plugin test artifacts for the MySQL matrix leg

PluginTests runs a MySQL + MariaDB matrix; the upload-artifacts condition
matched both legs and uploaded twice. Restrict the upload to the MySQL leg.

* Update expected UI screenshots for the new headless Chrome (Node 24 / Puppeteer 24) (#114)

* Update expected UI screenshots for the new headless Chrome (Node 24 / Puppeteer 24)

* Update expected UI screenshots for the new headless Chrome (Node 24 / Puppeteer 24)

* Re-sync expected UI screenshots for the new headless Chrome after rebase

Rendering drift only (identical content, minor line-height/AA shifts) under the
new headless Chrome used by the migrated screenshot-testing harness.

* Run the plugin UI tests on Node 24

Match the Node 24 / Puppeteer 24 screenshot-testing stack; the UI job was still pinned to Node 16.

* Update UI screenshots (#115)

Teal brand screenshots on 6.x-dev plugin base; synced from matomo-org/matomo run 30405694397.

* updating screenshots for sparkline feature-flag removal

* Update UI screenshots for the report header search

The DataTable search moved from the table footer into the shared ReportHeader
(matomo-org/matomo#24986), which changes every report screenshot.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

* Add changelog entry for the Matomo 6 release [ignore_release]

* Use the matomo6_* PHP version aliases in the test workflow [ignore_release]

The literals 8.1/8.5 drift as soon as core's supported range moves; the aliases
resolve centrally in matomo-org/github-action-tests.

* Update phpstan config for PHPStan 2 [ignore_release]

PHPStan 2 fails hard on a literal excludePaths entry that matches nothing. The
plugin ships no composer.json, so vendor/ never exists; github-action-tests/ only
exists during a CI run and is marked optional.

* Update UI screenshots for the collapsed report header search [ignore_release]

The report header search collapsed from an expanded input into an icon toggle,
which shifts every report table up by the height of the old input.

---------

Co-authored-by: Thomas ZILLIOX <thomas.zilliox@innocraft.com>
Co-authored-by: chippison <chip.pison@innocraft.com>
Co-authored-by: Altamash Shaikh <altamash@innocraft.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
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