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

AMP Validator: removing the word "just" from documentation #12115

Closed
adactio opened this issue Nov 18, 2017 · 4 comments
Closed

AMP Validator: removing the word "just" from documentation #12115

adactio opened this issue Nov 18, 2017 · 4 comments

Comments

@adactio
Copy link

@adactio adactio commented Nov 18, 2017

Hi,

The AMP Validator currently contains instructions inside an HTML comment.
https://validator.ampproject.org/

The comment reads:

This is the minimum valid AMP HTML document. Just type away here and the AMP Validator will re-check your document on the fly.

Suggested change—remove the word "just":

This is the minimum valid AMP HTML document. Type away here and the AMP Validator will re-check your document on the fly.

The reasoning is outlined here: http://bradfrost.com/blog/post/just/

“Just” makes me feel like an idiot. “Just” presumes I come from a specific background, studied certain courses in university, am fluent in certain technologies, and have read all the right books, articles, and resources. “Just” is a dangerous word.

Cheers,

Jeremy

@aghassemi
Copy link
Contributor

@aghassemi aghassemi commented Nov 20, 2017

@adactio Thanks for the suggestion. /to @Gregable

@honeybadgerdontcare
Copy link
Contributor

@honeybadgerdontcare honeybadgerdontcare commented Nov 20, 2017

We'll remove the word "just".

@honeybadgerdontcare
Copy link
Contributor

@honeybadgerdontcare honeybadgerdontcare commented Nov 20, 2017

This has now been released.

@adactio
Copy link
Author

@adactio adactio commented Nov 23, 2017

Thank you muchly!

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

Successfully merging a pull request may close this issue.

None yet
4 participants