AMP Mode Notice for Index listings

[postphotos]

As a user, I should be presented with a notice that reveals what mode of AMP I’m using so I understand what each action does based on the current mode.

Without this notice, a given user may think actions take effect on their site, but when auto-sanitize mode is turned on (either by force for "Native" mode or optionally for "Paired" mode),

• AC1: Surface a given notice based on the mode (from the settings page) in each of the views where actions can occur.
• AC2: Propose copy (and perhaps a suggestion to turn on Auto Suppression) for Native, Paired (non-auto) and Paired (auto) modes. Also, in Paired Mode (non-auto) offer a dismissable notice to help the user turn this on to enable auto sanitization sitewide.

Notes:

• If a user is in Paired Mode with auto-sanitize or in Native AMP mode, each error is but a notice and does not affect the site.
• If a user is in Paired Mode without auto-sanitize, each action might affect which pages are served as AMP. A given user, by not taking action on errors, would prevent AMP from being served on those pages.
• If a user is in Classic Mode, they should know they won't get the index listings (nor the single listings) as the sanitizer does not run in this mode.

[kienstra]

Question About Notice

Hi @postphotos,
Thanks for opening this issue.

Did you have in mind a notice like this one (that already exists):

Another point:

c) If a user is in Classic Mode, they should know they won't get the index listings (nor the single listings) as the sanitizer does not run in this mode.

In Classic Mode, the validator UI generally doesn't exist, including that screenshot above.

[kienstra]

AC1: Surface a given notice based on the mode (from the settings page) in each of the views where actions can occur.

Maybe there could also be a notice on the "AMP Validation Errors" page:

[postphotos]

Hi @kienstra, what you described here is what we have in mind for the scope of this ticket.

In short, currently a user might "accept" an error in Native AMP mode, but (lo and behold) nothing happens. Between showing an icon (i.e. the AMP lightning bolt logo) next to AMP serving URLs, and limited (perhaps dismissable? @jwold) notices on Index Pages we hope to reduce the confusion on what actions mean for index pages. Instead of the click-and-explore method, we want to give them clear notices for what they should expect. I know @jwold has a sketch that explains the possible states.

I broke apart AC2 here - We've worked on copy a bit already, but you should be unblocked to start AC1 at your leisure.

[miina]

@postphotos To confirm:

1. Would the notice be on all index pages (as of now on the Errors page and on the Invalid Pages both)?
2. Is this the notice that the issue involves:
   

Edit:
As of now added the following notices, let me know if you already had some texts:

• Native mode:
  The site is using native AMP mode, the validation errors found are already automatically handled.
• Paired mode with auto-sanitization:
  The site is using paired AMP mode with auto-sanitization turned on, the validation errors found are already automatically handled.
• Paired mode without auto-sanitization:
  The site is using paired AMP mode without auto-sanitization, the validation errors found require action and influence which pages are shown in AMP. For automatically handling the errors turn on auto-sanitization from Validation Handling settings. (<--- link)
• Classic mode:
  The site is using classic AMP mode, your theme templates are not used and the errors below are irrelevant.
  <--- Although the index pages are not visible when in classic mode, it is possible to see this notice when turning on Classic mode in one tab and refreshing an index page in the other tab, thus the notice.

For example:

[westonruter]

I don't think it needs to be dismissible, does it?

[miina]

True, it doesn't. Removed.

[postphotos]

Hi @miina, thanks for this update. I have no issue with the given wording that's been written, and think this is the intention of this ticket. 👍 Looks good to me!

[kienstra]

Request For Testing

Hi Claudio,
Could you please test the notices, like the on the Invalid URLs page?

[csossi]

Notices appearing for Invalid URLS index and Error Index