Skip to content
This repository has been archived by the owner on Mar 29, 2021. It is now read-only.

AMP support #604

Open
mmprep opened this issue Feb 18, 2020 · 5 comments
Open

AMP support #604

mmprep opened this issue Feb 18, 2020 · 5 comments

Comments

@mmprep
Copy link

mmprep commented Feb 18, 2020

Hi Derek,

Your plugin is a brilliant, thank you very much!.

However, there is one problem: it doesn’t work on AMP pages. But Google AMP technology, designed to speed up the mobile web, is very and very popular. Moreover, the speed of the site is one of the key factors in its ranking.

Question: when do you plan to implement AMP support?

--
Dmitry

@dsifford
Copy link
Owner

Hi @mmprep... What specifically is not working on AMP? I assume the tooltips? That's not anything I can fix I'm afraid.

@mmprep
Copy link
Author

mmprep commented Feb 18, 2020

The tooltips and bibliography list view/hide are not working in AMP.
I would like to know why it is impossible to implement AMP support?

@dsifford
Copy link
Owner

By design, AMP does not run user-provided javascript. To be fast, AMP only takes the rendered content of pages and serves that. Those things depend on javascript to function.

@mmprep
Copy link
Author

mmprep commented Feb 18, 2020

OK, but what about https://amp.dev/documentation/guides-and-tutorials/develop/custom-javascript/ and probably CSS methods?

@dsifford
Copy link
Owner

I'd have to investigate this more. Scanning that quickly, it still looks like there's some limiting factors that may outright block this from functioning even after following their rules (js size, run time, etc).

No idea as to when I would even be able to get around to this unfortunately.

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

No branches or pull requests

2 participants