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

Add 'More details' link to 'Status' tooltip #1670

Closed
wants to merge 1 commit into from

Conversation

kienstra
Copy link
Contributor

This adds a 'More details' link for the 'Status' tooltips on 3 of the 4 Compatibility Tool pages:

  • AMP Validated URLs
  • Single URL page
  • AMP Validation Error Taxonomy

new-link-more-details

Closes #1400

Add a link that further explains
AMP validtion.
@kienstra
Copy link
Contributor Author

Request For Review

Hi @westonruter,
Could you please review this? Thanks!

@kienstra
Copy link
Contributor Author

Delaying until links are live

As the links in this are to a site that's not deployed yet, I'm closing this PR, probably until after the 1.0 stable release.

Feel free to reopen it if you'd like.

@kienstra kienstra closed this Nov 30, 2018
@kienstra kienstra mentioned this pull request Nov 30, 2018
4 tasks
@schlessera schlessera deleted the update/1400-tooltip-links branch March 7, 2020 06:25
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.

1 participant