-
Notifications
You must be signed in to change notification settings - Fork 11
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
PDF Support on published graphs #19
Comments
I tested the export in publish-spa (with a 0.9.18 static build) with same issue and my workaround doing the basic job. publish-spa main was hopefully updated to 0.3.0 by pulling main and running just Now I want to update the static build to 0.10.x first and then investigate publish-spa deeper. |
Is there any update on this? Would be great to have PDFs linked in the published page. Is the team at least considering this issue? |
Currently, published graphs that link a PDF don't work. Not only you can't use the built-in logseq viewer, you can't even download the PDF (without editing the URL or using your browser's developer tools). It's as if the link is broken.
An easy meantime patch could be just linking the PDF so it works in the browser or it downloads the file, even if highlighting and notes don't work.
As @LuciferianInk said:
To Reproduce
Steps to reproduce the behavior (from original post logseq/logseq#4048):
Workaround
Manually change the Link in the Markdown source of the page into a file asset:
Remove the leading
!
and replace the two dots starting the go-one-up asset link with a same level file link prefix[Filename-of-the-pdf-as-text.pdf](file://./assets/Filename-of-the-pdf-as-text.pdf)
Solution to finalize the workaround
![Name.pdf](../assets/Name.pdf)
Link in the MD[Name.pdf](file://./assets/Name.pdf)
just after the regular PDF asset linkThis exports the PDF into assets, works as usual in the Logseq app but adds at least link access to the asset!
Poor but better than nothing.
Note
This is a duplicate of logseq/logseq#4048
The text was updated successfully, but these errors were encountered: