Skip to content

Improve the first-dashboard launch banner - #6535

Open
sanne-san wants to merge 11 commits into
verification-to-dashboardfrom
sanne-email-banner
Open

Improve the first-dashboard launch banner#6535
sanne-san wants to merge 11 commits into
verification-to-dashboardfrom
sanne-email-banner

Conversation

@sanne-san

Copy link
Copy Markdown
Contributor

Changes

  • Improve the design and copy of the banner using the generic notice component
  • Trigger when the site has pageviews and only for roles with email-reports access, instead of a session-scoped flag
  • Ensure the banner is hidden while the verification banner is up, and revealed on verification-finished
  • Persist dismissal per user + site when X button or CTA click is clicked

Tests

  • Automated tests have been added

Dark mode

  • The UI has been tested both in dark and light mode

RobertJoonas and others added 11 commits July 20, 2026 09:49
* Refine verification banner UI and copy

- Simplify failure CTAs: keep "Check again" as primary, add ghost
  "Review installation" (or "Try another URL" when a custom URL retry
  is offered); remove installation-guide, change-method, and view-snippet
  expandable logic
- Render inline "verify your installation manually" / "review your
  installation" links inside recommendation text (offer_custom_url_input
  scenarios) via safe HTML helpers, avoiding HEEx whitespace pitfalls
- Tighten diagnostics title and body copy
- Move the "Setup pending" pill in the sites list to replace the
  percentage indicator; use the generic pill component
- Extend the notice component with title_class override and a spinner
  slot; swap success icon to solid check-circle

* improve templating logic

---------

Co-authored-by: Robert Joonas <robertjoonas16@gmail.com>
- Improve the design and copy of the banner using the generic notice component
- Trigger when the site has pageviews and only for roles with email-reports access, instead of a session-scoped flag
- Ensure the banner is hidden while the verification banner is up, and revealed on `verification-finished`
- Persist dismissal per user + site when X button or CTA click is clicked
@sanne-san
sanne-san requested a review from RobertJoonas July 22, 2026 16:46
@RobertJoonas
RobertJoonas force-pushed the verification-to-dashboard branch 2 times, most recently from 45547c4 to 3fd5593 Compare July 29, 2026 11:03
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