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 "View AMP" link to admin bar on non-AMP page in Reader mode #2203

Closed
westonruter opened this issue Apr 26, 2019 · 0 comments · Fixed by #2227
Closed

Add "View AMP" link to admin bar on non-AMP page in Reader mode #2203

westonruter opened this issue Apr 26, 2019 · 0 comments · Fixed by #2227
Milestone

Comments

@westonruter
Copy link
Member

westonruter commented Apr 26, 2019

A very common support request is confusion about why adding /amp/ to the URL leads to a 404 instead. This is normally due to trying to access the AMP version of a Page or some other hierarchical post type, for which ?amp is used for the AMP URL. (See also #2062 to automatically redirect from /amp/ to ?amp when is_404().) When users do not have the AMP Validator extension installed, they will not see the AMP link in the browser extension button. If users are unfamiliar with the amphtml link in the head, they will have no idea how to find the AMP version or even if there is an AMP version available.

So we should consider adding the AMP link to the admin bar for AMP-enabled posts, even in reader mode:

image

Aside: We should consider eliminating the /amp/ in favor of ?amp everywhere. The endpoint slug is unnecessary for SEO, and it improve consistency when switching between the paired modes (transitional and reader). Additionally, there will no longer be a need to flush rewrite rules. See #2204.

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