Skip to content

The fragment-index Has No Effect on Images for Reveal.JS #3539

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

Open
4 tasks done
RoyiAvital opened this issue Dec 1, 2022 · 5 comments
Open
4 tasks done

The fragment-index Has No Effect on Images for Reveal.JS #3539

RoyiAvital opened this issue Dec 1, 2022 · 5 comments
Labels
enhancement New feature or request revealjs Issues with the revealjs format
Milestone

Comments

@RoyiAvital
Copy link

RoyiAvital commented Dec 1, 2022

Bug description

It seems that Quarto doesn't apply the fragment-index to images:

## How to Measure

::: {.incremental}
- Assume we have 2 variants of a website: `A` and `B`.
- How would we measure their design effect on user engagement?
:::

![](https://i.imgur.com/PSZggyN.png){.absolute bottom=0 .fragment .fade-in-then-out fragment-index=20}

The output will have data-fragment-index=0 for the image.

In the case above I want the image to appear in sync with an item of the list.
So I'd like to be able to have ![](https://i.imgur.com/PSZggyN.png){.absolute bottom=0 .fragment .fade-in-then-out fragment-index=20} yet what ever I do, I can't control its index.

Just as a note, it would be great to be able to have control on the index in .incremental. Maybe something like defining the index of the first item (Then all are chained)?

This is tested on Windows 10, VS Code, Quarto Extension 1.56 and Quarto CLI 1.2.269.

Checklist

  • Please include a minimal, fully reproducible example in a single .qmd file? Please provide the whole file rather than the snippet you believe is causing the issue.
  • Please format your issue so it is easier for us to read the bug report.
  • Please document the RStudio IDE version you're running (if applicable), by providing the value displayed in the "About RStudio" main menu dialog?
  • Please document the operating system you're running. If on Linux, please provide the specific distribution.
@RoyiAvital RoyiAvital added the bug Something isn't working label Dec 1, 2022
@cscheid cscheid added enhancement New feature or request and removed bug Something isn't working labels Dec 1, 2022
@cscheid cscheid added this to the Future milestone Dec 1, 2022
@cscheid
Copy link
Collaborator

cscheid commented Dec 1, 2022

Incremental is a Pandoc feature, and it's a feature of lists specifically: https://pandoc.org/MANUAL.html#incremental-lists

We would have to intercept and reimplement that entire infrastructure to enable this. It'd be a great one to have, but it's not in our short-to-medium-term radar.

@RoyiAvital
Copy link
Author

OK, I get that we won't be able to make the {.incremental} thing be explicit (Unless we ask pandoc guys?).

But the image losing its index is the bug I'm after.
Is there a way to force the index on the image?

@cscheid
Copy link
Collaborator

cscheid commented Dec 1, 2022

That image isn't part of the incremental list, so I think what you're suggesting can't work. You might have better luck with pauses (. . .), as explained here.

@RoyiAvital
Copy link
Author

Leave alone the effect I want to achieve. I get it is not achievable as things now (What I do is manually add indices to the elements).
The image itself doesn't retain the index set to it. That's the bug I reported.

@RoyiAvital
Copy link
Author

@cscheid , I created #3559.
Could you mark this as a bug and only pay attention to the issue of the fragment-index of the image?

@cderv cderv added the revealjs Issues with the revealjs format label May 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request revealjs Issues with the revealjs format
Projects
None yet
Development

No branches or pull requests

3 participants