Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Validator Chrome extension to show debug icon instead of error when in dev mode #24176

Closed
westonruter opened this issue Aug 23, 2019 · 3 comments · Fixed by #30011
Closed

Comments

@westonruter
Copy link
Member

In #20974 a new data-ampdevmode attribute was added which tells the validator to suppress reporting validation errors for elements it appears on, except for the root html element:

image

In such case, however, when there is only one validation error and it is due to being in debug mode, it would be very useful if the Validator extension would not show an error icon but rather a debug icon, similar to how it shows a warning icon when there are deprecation issues.

If there are other validation errors other than the one data-ampdevmode on the root html element, then the Validator extension should continue to show the error icon.

By showing a debug icon, users won't get alarmed when the system is generating markup that is explicitly invalid for dev mode purposes.

Originally suggested in #20974 (comment).

@westonruter
Copy link
Member Author

@honeybadgerdontcare When do you think this will be implemented? We're planning to release the next version of the WordPress AMP plugin in a couple weeks and it will have support for dev mode. We'll want to include an explanation of this mode and what users can expect when using the AMP validator extension. So having a sense of timing will help.

@honeybadgerdontcare
Copy link
Contributor

I have no timeline for implementing this, there are other high priority items I'm working on.

@westonruter
Copy link
Member Author

We are starting to get more AMP plugin user reports about this. For example: https://wordpress.org/support/topic/amp-dev-mode-error-with-amp-validator/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants