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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Global variable links clickable in PDF exports #2450

Merged
merged 2 commits into from
Jan 19, 2023
Merged

Conversation

fonsp
Copy link
Owner

@fonsp fonsp commented Jan 19, 2023

This PR will use tag for global variable links. This makes global variable links clickable in PDF exports 馃帀

Video below is my PDF viewer, reading a PDF generated by Pluto:

Schermopname.2023-01-19.om.19.50.48.mov

We initially used <a>, but because of #1790 we changed it to <pluto-variable-link>. I noticed that we can change it back to <a> without getting #1790 again, this was probably fixed in the codemirror 6.0 release. (cc @dralletje )

TODO:

@github-actions
Copy link
Contributor

Try this Pull Request!

Open Julia and type:

julia> import Pkg
julia> Pkg.activate(temp=true)
julia> Pkg.add(url="https://github.com/fonsp/Pluto.jl", rev="a-tag-for-variable-links")
julia> using Pluto

@fonsp fonsp marked this pull request as ready for review January 19, 2023 19:04
@fonsp fonsp linked an issue Jan 19, 2023 that may be closed by this pull request
@fonsp fonsp merged commit 50ead60 into main Jan 19, 2023
@fonsp fonsp deleted the a-tag-for-variable-links branch January 19, 2023 23:24
@fonsp fonsp changed the title Use <a> tag for global variable links Global variable links clickable in PDF exports Jan 20, 2023
@fonsp fonsp added frontend Concerning the HTML editor CM6 CodeMirror 6 labels Jan 30, 2023
@fonsp fonsp added the publishing Notebooks as static documents on the web label Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CM6 CodeMirror 6 frontend Concerning the HTML editor publishing Notebooks as static documents on the web
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Suppress more interactive elements in PDF/print export
1 participant