Skip to content

Add paper-spin extension for RevealJS titles - #2149

Open
nickriches wants to merge 2 commits into
quarto-dev:mainfrom
nickriches:patch-1
Open

Add paper-spin extension for RevealJS titles#2149
nickriches wants to merge 2 commits into
quarto-dev:mainfrom
nickriches:patch-1

Conversation

@nickriches

Copy link
Copy Markdown

Added a new Quarto Extension for a newspaper-style spin-in effect.

Added a new Quarto Extension for a newspaper-style spin-in effect.

@cderv cderv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you correct link of repo so we could have look ?

thank you

Comment thread docs/extensions/listings/shortcodes-and-filters.yml Outdated
@nickriches

nickriches commented Aug 28, 2026 via email

Copy link
Copy Markdown
Author

Comment thread docs/extensions/listings/shortcodes-and-filters.yml Outdated

@cderv cderv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the submission!

One thing to fix before this can merge:

  • Listing name mismatch: the entry is named paper-spin, but the repository is quarto-paper-pile, the extension installs as _extensions/nickriches/paper-pile, and the shortcode is {{< paper-pile ... >}}. Could you rename the listing entry to paper-pile to match, or clarify why a different name is intended?

Separately, unrelated to this PR: quarto add nickriches/quarto-paper-pile (the exact command in your README) currently fails with "Extension not found in local or remote sources". This turned out to be a quarto-cli bug — it only ever tries the main branch and never falls back to master, which is this repo's default branch (filed as quarto-dev/quarto-cli#14821). Until that's fixed upstream, quarto add nickriches/quarto-paper-pile@master is the working install command — might be worth
noting in your README in the meantime, or change your repo default branch to main. The later is better if possible.

Happy to take another look once the naming is sorted out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants