Summary

Fixes #2023.

• Rebrand “accepting” the sanitization the invalid markup causing a validation error to rather be “removing” the invalid markup. Similarly, rebrand “reject” as “keep”. As such, “Status” generally now becomes “Markup Status”
• Use different colors of AMP logo as icons for the various states, as opposed to the many different icons previously used (e.g. ✅ and ❌).
• Improve descriptions of stylesheet error codes to indicate they relate to CSS.

Validated URLs List

• Include whether AMP is enabled or disabled in the list.
• Combine the counts of new errors into the remove/keep counts.
• Rename “Invalid” column to “Invalid Markup”
• Remove red color from Invalid Markup column.
• Improve sources summary (see section below).
• Update filter dropdowns to refer to actions taken on invalid markup rather than validation errors sanitization statuses.

Before	After

Validation Error Index

• Per above, use “Removed” instead of “Accepted” and “Kept” instead of “Rejected”.
• Per above, use only red and green AMP logos to indicate whether the invalid markup is kept or removed, respectively.

Before	After

Validation Error Detail

• Use green/red buttons to correspond to the removed/kept actions.
• Include “confirm” in the action buttons/links for remove/keep when in the new state, as removing already-removed invalid markup just clears the new state.

Before	After

Validated URL with Kept Markup (Rejected Error)

• Re-work the page heading to just say “AMP Validated URL”, with subheading for the actual title of the queried object (if available).
• Use same summary from validated URL list in the status metabox.

Before	After

Validated URL with Errors in all 4 states

• As above, re-work the page heading to just say “AMP Validated URL”, with subheading for the actual title of the queried object (if available).
• Eliminate separate dropdown options for the “New” states, keeping the choices to just: Removed and Kept.
• As above, merge new error counts into the total counts for invalid markup removed/kept.

Before	After

Improvement to sources summary

Previously when there was a validation error occurring in the content, the summary would show:

Now, however, a translatable “Content” label is used:

And, more importantly, when there is more specific information available, like the block name, it is used instead:

Additionally, if Gutenberg is one of the sources, it will be suppressed since it should be considered the same as core.

Checklist

• My pull request is addressing an open issue (please create one otherwise).
• My code is tested and passes existing tests.
• My code follows the Engineering Guidelines (updates are often made to the guidelines, check it out periodically).