-
Notifications
You must be signed in to change notification settings - Fork 397
feat(reveal.js): add support for fragment indices in line highlighting #13125
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
base: main
Are you sure you want to change the base?
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
|
Thanks for initiating ! Could be also related to
Also, do we have an issue about this new feature with a discussion or only the Slack question? And regarding naming, I think I would like to handle this one too before adding new options Just listing all this as thoughts for later. |
|
Thanks for linking. The changes would indeed fix both issues. (I'm not linking them formally at this point to avoid risking closing the issues) The PR is mostly here to not lose the code. |
|
I updated the PR for it to be more complete: changelog, test, etc. |
Introduce support for fragment indices to enhance line highlighting functionality, allowing for more precise control over highlighted code segments in Reveal.js.
This adds a
code-line-fragment-indicesattribute to offer control over fragment.When set, the same number of fragment indices as there are steps are required.
I'm not sure of the other changes that this new feature should be coming with.