Skip to content

fix: ensure some slides are not visible on print (meanwhile we remove them, reveal add them elsewhere...) - #82

Merged
kuroidoruido merged 1 commit into
mainfrom
fix/extra-slide-on-print-multi-cols
Jan 29, 2026
Merged

kuroidoruido merged 1 commit into
mainfrom
fix/extra-slide-on-print-multi-cols

Conversation

@kuroidoruido

Copy link
Copy Markdown
Contributor

As I analyzed:

  • multiple cols feature create some slide duplication, but some with no content
  • the actual behavior is to remove those slides but it does not remove everything (we remove every slide we can get, but some slides seems to be added at another timing)
  • I did not understand for now why there is those extra slides, but adding an attribute works so...
  • I added a css hack to hide every slide with attribute data-slide-to-remove and it "works"

@netlify

netlify Bot commented Jan 29, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for talk-control-revealjs-extensions ready!

Name Link
🔨 Latest commit f339fdc
🔍 Latest deploy log https://app.netlify.com/projects/talk-control-revealjs-extensions/deploys/697b842e2ed45e0008dbb24e
😎 Deploy Preview https://deploy-preview-82--talk-control-revealjs-extensions.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@kuroidoruido
kuroidoruido force-pushed the fix/extra-slide-on-print-multi-cols branch from 076746a to f339fdc Compare January 29, 2026 16:00
@kuroidoruido
kuroidoruido merged commit 08c47e6 into main Jan 29, 2026
5 checks passed
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.

1 participant