Skip to content

Update expected UI screenshots for the cloud blocking settings split [ignore_release] - #339

Open
snake14 wants to merge 5 commits into
6.x-devfrom
pg-5445-cloud-blocking-mode
Open

Update expected UI screenshots for the cloud blocking settings split [ignore_release]#339
snake14 wants to merge 5 commits into
6.x-devfrom
pg-5445-cloud-blocking-mode

Conversation

@snake14

@snake14 snake14 commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Description

The TrackingSpamPrevention section on the general settings page changed height in matomo-org/matomo#25208, which splits its single "block tracking requests from the cloud" tickbox into a three-way organisation blocking mode and a separate cloud IP range tickbox. This plugin's settings card sits below that section on the same page, so the point at which its screenshot is captured shifts by a pixel and the anti-aliasing follows. No behaviour of this plugin changes.

The image was taken from the core pull request's CI run rather than generated locally.

Issue No

PG-5445

Steps to Replicate the Issue

  1. Run this plugin's UI tests against Split cloud hosting provider blocking into two settings, PG-5445 matomo#25208.
  2. Expected: the settings screenshot matches.
  3. Actual: it differs by the capture offset described above.

Checklist

  • [✔] Tested locally or on demo2/demo3?
  • [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?

…[ignore_release]

The TrackingSpamPrevention section on the general settings page changed height in
matomo-org/matomo#25208, which shifts where this card is captured by a pixel.
…king-mode

# Conflicts:
#	tests/UI/expected-ui-screenshots/QueuedTrackingSettings_settings_page.png
#	tests/UI/expected-ui-screenshots/QueuedTrackingSettings_settings_page_sentinel.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.

1 participant