Skip to content

Update the view-access-restricted screenshot for the refreshed login layout - #1159

Open
chippison wants to merge 1 commit into
prepare6xfrom
dev-20463-redesign
Open

Update the view-access-restricted screenshot for the refreshed login layout#1159
chippison wants to merge 1 commit into
prepare6xfrom
dev-20463-redesign

Conversation

@chippison

Copy link
Copy Markdown
Contributor

Description

Updates the view-access-restricted baseline for the refreshed login layout in matomo-org/matomo#24998. That page renders the sign-in form with an error notification, and #24998 replaces the centred-card-under-a-top-bar shell with the refreshed single column, so the previous baseline still captured the old chrome.

Cut from prepare6x rather than stacked on #1158, so it can be reviewed and merged on its own. The PNG is replaced wholesale, so it does not depend on #1158's intermediate update. When #1158 merges into prepare6x this branch will need a rebase, keeping this version of the baseline.

The PNG is the /processed/ output of Matomo core CI run 30970737938, verified byte-identical by SHA-256 against the artifact rather than rendered locally.

Heads up on CI: the UI check here will stay red until matomo-org/matomo#24998 merges, because the baseline describes login markup that is not yet in the core branch this plugin's CI builds against. #1158 is currently red for exactly the same reason.

Issue No

DEV-20463

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?
  • [NA] I have understood, reviewed, and tested all AI outputs before use
  • [NA] All AI instructions respect security, IP, and privacy rules
  • [NA] Documentation updated?

AltamashShaikh
AltamashShaikh previously approved these changes Aug 6, 2026

@AltamashShaikh AltamashShaikh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chippison
chippison force-pushed the dev-20463-redesign branch 2 times, most recently from 2a2494a to a7ef555 Compare August 10, 2026 00:21
@tzi
tzi changed the base branch from prepare6x to 6.x-dev August 12, 2026 01:20
@tzi
tzi dismissed AltamashShaikh’s stale review August 12, 2026 01:20

The base branch was changed.

@tzi
tzi changed the base branch from 6.x-dev to prepare6x August 12, 2026 01:20
@chippison

Copy link
Copy Markdown
Contributor Author

Should be good to be reviewed and merged into prepare6x

@AltamashShaikh AltamashShaikh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chippison There are other UI screenshots which are failing too, unrelated to this one.

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