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 "visit site" link to top of AMP content #542

Closed
gibrown opened this issue Oct 17, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@gibrown
Copy link

commented Oct 17, 2016

A general concern is that it is not possible for users to get back to the original site. The header does do this, but is not very clear. (https://www.alexkras.com/google-may-be-stealing-your-mobile-traffic/)

A similar issue exists in the WP.com Reader, and we're adding more prominent visit links into the display: Automattic/wp-calypso#8791

Though I admit, maybe this should not be implemented within the AMP page.

Showing recent/related content maybe solves the same problem: #124 (comment)

@amedina

This comment has been minimized.

Copy link
Member

commented Jul 6, 2017

I think you are correct: this should be implemented as part of the AMP page and not inserted by the plugin.

@amedina amedina closed this Jul 6, 2017

@gibrown

This comment has been minimized.

Copy link
Author

commented Jul 6, 2017

@amedina should we keep this open to track it? This seems to be a constant thing that comes up with sites in Google search.

2017-07-06 07 33 18

The header gets hidden, and even if you can see the header, then it is not immediately obvious that it is a link. To some extent this is Google's problem, but I think our default page could be a lot better and address this.

@amedina

This comment has been minimized.

Copy link
Member

commented Jul 6, 2017

Hi @gibrown; I agree this is an important issue. It falls into the more general goal of improving the quality of the templates used by the plugin in paired mode (i.e. AMP vs. non-AMP). I opened an issue for that and referenced this one there. Let's track both together there.

@gibrown

This comment has been minimized.

Copy link
Author

commented Jul 7, 2017

Sounds great, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.