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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(revealjs): disable menu open button #260

Merged
merged 2 commits into from Feb 2, 2023
Merged

feat(revealjs): disable menu open button #260

merged 2 commits into from Feb 2, 2023

Conversation

sagikazarmark
Copy link
Contributor

This is a workaround for denehyg/reveal.js-menu#99

When the RevealMenu plugin is enabled, there is a menu button on the printed slides.

This change disables that menu during printing.

This isn't necessarily the best possible solution, but since there is no "official" API for updating plugin configuration and the RevealMenu plugin does not expose any destroy or hide button API, this is the best alternative at the moment.

Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
Copy link
Owner

@astefanutti astefanutti left a comment

Choose a reason for hiding this comment

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

LGTM. Adding a comment would be great, and it'll be good to go 👍🏼.

plugins/reveal.js Show resolved Hide resolved
Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
@astefanutti astefanutti merged commit c92702b into astefanutti:master Feb 2, 2023
@astefanutti
Copy link
Owner

Thanks!

@sagikazarmark sagikazarmark deleted the revealjs-menu branch February 2, 2023 13:28
@sagikazarmark
Copy link
Contributor Author

Thanks for quickly merging it!

@sagikazarmark
Copy link
Contributor Author

@astefanutti would you consider releasing a new version with this fix? Thanks!

@astefanutti
Copy link
Owner

@sagikazarmark thanks for the reminder. I've just released version 3.6.0 that contains this fix. Thanks again!

@sagikazarmark
Copy link
Contributor Author

Awesome, thanks!

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.

None yet

2 participants