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

Jquery needs to be included by the theme so the floating footer works #4739

Closed
choldgraf opened this issue Oct 8, 2018 · 9 comments
Closed
Labels
Needed: design decision A core team decision is required

Comments

@choldgraf
Copy link
Contributor

choldgraf commented Oct 8, 2018

Details

Expected Result

The documentation is built as PDF/EPUB (since those boxes are checked on my advanced settings menu). Also, I expect to see the little floating window in the bottom-right since I'm hosting on readthedocs.

Actual Result

There is no built file in /downloads, nor in the downloads tab on the RTD config page. There is also no floating box.

@stsewd
Copy link
Member

stsewd commented Oct 8, 2018

PDF and epub formats aren't supported by MkDocs. The floating box is an issue, though.

@stsewd
Copy link
Member

stsewd commented Oct 8, 2018

Looks like jquery is missing when injecting the rtd's version selector (floating box)

@stsewd stsewd added the Support Support question label Oct 8, 2018
@choldgraf
Copy link
Contributor Author

ahh - it'd be helpful if it were made clear that PDF generation only works for Sphinx builds and not mkdocs...perhaps here :-)

re: the floating box, do we need to specify jquery as a dependency somehow?

@stsewd
Copy link
Member

stsewd commented Oct 8, 2018

We mention that here https://docs.readthedocs.io/en/latest/builds.html#mkdocs, I'm not sure if we need to mention that on the features pages, as it's a MkDocs limitation, not rtd.

re: the floating box, do we need to specify jquery as a dependency somehow?

I think we inject those dependencies somewhere (or maybe we assume that everyone is using jquery?) I need to check the code.

@choldgraf
Copy link
Contributor Author

Sure - I'm just giving you some data from my personal path through that documentation...I didn't notice that statement about PDF on that page

@stsewd
Copy link
Member

stsewd commented Oct 22, 2018

So, I looked into the code, we inject the code for the floating footer https://github.com/rtfd/readthedocs.org/blob/a4291d17728b8df6b7a9ce57537c5d620416e485/readthedocs/doc_builder/backends/mkdocs.py#L106-L106, but we don't have jquery as an explicit dependency https://github.com/rtfd/readthedocs.org/blob/master/readthedocs/core/static-src/core/js/readthedocs-doc-embed.js

Not sure if we should add jquery or rely on themes to added it. Going to mark this as a design decision.

@stsewd stsewd added Needed: design decision A core team decision is required and removed Support Support question labels Oct 22, 2018
@stsewd stsewd changed the title PDF/EPUB generation doesn't seem to be happening. Jquery needs to be included by the theme so the floating footer works Oct 22, 2018
@humitos
Copy link
Member

humitos commented Jan 7, 2019

The original project with the issue is no longer public. Also, the floating menu is working for an MkDocs project (https://mkdocs.readthedocs.io/en/stable/) without problem.

I'm closing the issue here. Feel free to reopen if you consider.

@humitos humitos closed this as completed Jan 7, 2019
@choldgraf
Copy link
Contributor Author

That's fine, though just a quick note that the project is still public, we just corrected a typo in the url

https://bids-specification.readthedocs.io/en/latest/

@stsewd
Copy link
Member

stsewd commented Jan 8, 2019

@humitos I was thinking that we could bundle the footer with all the requirements (only jquery here I think), so it works in every theme.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needed: design decision A core team decision is required
Projects
None yet
Development

No branches or pull requests

3 participants