Skip to content

Update expected UI screenshots for the restyled selects and drag handle, PG-942 - #274

Draft
snake14 wants to merge 4 commits into
prepare6xfrom
PG-942-fix-draggable-list-handle
Draft

Update expected UI screenshots for the restyled selects and drag handle, PG-942#274
snake14 wants to merge 4 commits into
prepare6xfrom
PG-942-fix-draggable-list-handle

Conversation

@snake14

@snake14 snake14 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Description

Companion to matomo-org/matomo#25033 (restyled expandable selects and the standard drag handle): updates this plugin's 4 expected UI screenshots to the new component rendering. The screenshots were generated by that PR's CI and synced from its artifacts, then reviewed individually — the diffs are the restyled components only.

This branch is cut from the commit matomo core's 6.x-dev currently pins for this plugin (one commit ahead of the prepare6x merge-base), because that pin is what core CI renders; the core PR advances its gitlink to this branch. Targeted at prepare6x as the 6.x line; the small base divergence is pre-existing pin drift.

Merge-order dependency: this PR's own UI check tests against plain 6.x-dev and fails on exactly the synced screenshots until matomo-org/matomo#25033 merges — the new expectations render only with that PR's restyle. Merge matomo#25033 first, then this.

Issue No

Related to matomo-org/matomo#25033 (PG-942)

Steps to Replicate the Issue

  1. Run Restyle expandable select dropdowns and add a standard drag handle component matomo#25033's UI-plugins CI job for this plugin.
  2. Expected: screenshots match the restyled expandable selects and drag handle.
  3. Actual before this change: 4 screenshot comparisons failed against the pre-restyle expectations.

Checklist

  • [✖] Tested locally or on demo2/demo3? (validated via Restyle expandable select dropdowns and add a standard drag handle component matomo#25033's CI and visual review of every synced screenshot)
  • [NA] New test case added/updated?
  • [NA] Are all newly added texts included via translation?
  • [NA] Are text sanitized properly? (Eg use of v-text v/s v-html for vue)
  • [NA] Version bumped?
  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules
  • [NA] Documentation updated?

tzi and others added 4 commits July 29, 2026 01:31
Teal brand screenshots on 6.x-dev plugin base; synced from matomo-org/matomo run 30405694397.
…d drag handle

Generated against the commit matomo core's 6.x-dev currently pins, by matomo-org/matomo#25033's CI run.
…able-list-handle

# Conflicts:
#	tests/UI/expected-ui-screenshots/CustomAlerts_alert_condition_reloaded_site1.png
#	tests/UI/expected-ui-screenshots/CustomAlerts_alert_condition_reloaded_site2.png
#	tests/UI/expected-ui-screenshots/CustomAlerts_edit.png
#	tests/UI/expected-ui-screenshots/CustomAlerts_report_mediums.png
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.

2 participants