Skip to content

swap reveal.js instead of swiper for slides#9166

Draft
Light2Dark wants to merge 4 commits intomainfrom
sham/add-reveal-js
Draft

swap reveal.js instead of swiper for slides#9166
Light2Dark wants to merge 4 commits intomainfrom
sham/add-reveal-js

Conversation

@Light2Dark
Copy link
Copy Markdown
Collaborator

@Light2Dark Light2Dark commented Apr 13, 2026

📝 Summary

Swap libraries from Swiper to Reveal.js for slides view.

Screen.Recording.2026-04-14.at.12.33.05.AM.mov

Todo:

  • Update pdf export flows
  • Read mode

📋 Pre-Review Checklist

  • For large changes, or changes that affect the public API: this change was discussed or approved through an issue, on Discord, or the community discussions (Please provide a link if applicable).
  • Any AI generated code has been reviewed line-by-line by the human PR author, who stands by it.
  • Video or media evidence is provided for any visual changes (optional).

✅ Merge Checklist

  • I have read the contributor guidelines.
  • Documentation has been updated where applicable, including docstrings for API changes.
  • Tests have been added for the changes made.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
marimo-docs Ready Ready Preview, Comment Apr 14, 2026 4:44am

Request Review

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2026

Bundle Report

Changes will increase total bundle size by 205.84kB (0.83%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
marimo-esm 25.05MB 205.84kB (0.83%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: marimo-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/cells-*.js -74 bytes 703.41kB -0.01%
assets/index-*.js -14.87kB 587.28kB -2.47%
assets/index-*.css 63 bytes 362.35kB 0.02%
assets/dist-*.js 56 bytes 58.37kB 0.1%
assets/JsonOutput-*.js 471 bytes 336.76kB 0.14%
assets/ai-*.js 385 bytes 250.2kB 0.15%
assets/esm-*.js -5 bytes 9.78kB -0.05%
assets/layout-*.js -309 bytes 185.6kB -0.17%
assets/cell-*.js 21 bytes 182.93kB 0.01%
assets/reveal-*.js (New) 165.48kB 165.48kB 100.0% 🚀
assets/swiper-*.js (New) 116.4kB 116.4kB 100.0% 🚀
assets/reveal-*.css (New) 53.96kB 53.96kB 100.0% 🚀
assets/swiper-*.css (New) 15.12kB 15.12kB 100.0% 🚀
assets/mermaid-*.core-CIrjnvnv.js (New) 2.38kB 2.38kB 100.0% 🚀
assets/slide-*.js (New) 615 bytes 615 bytes 100.0% 🚀
assets/slides-*.css -15.1kB 305 bytes -98.02%
assets/slides-*.js -116.37kB 0 bytes -100.0%
assets/mermaid-*.core-CvPV98yl.js (Deleted) -2.38kB 0 bytes -100.0% 🗑️

Files in assets/index-*.js:

  • ./src/plugins/layout/carousel/CarouselPlugin.tsx → Total Size: 692 bytes

Files in assets/layout-*.js:

  • ./src/components/editor/renderers/slides-layout/slides-layout.tsx → Total Size: 2.87kB

Files in assets/reveal-*.js:

  • ./src/components/slides/reveal-slides.css → Total Size: 0 bytes

  • ./src/components/slides/reveal-component.tsx → Total Size: 3.78kB

Files in assets/swiper-*.js:

  • ./src/components/slides/swiper-component.tsx → Total Size: 5.4kB

  • ./src/components/slides/swiper-slides.css → Total Size: 0 bytes

Files in assets/slides-*.js:

  • ./src/components/slides/slides.css → Total Size: 0 bytes

@Light2Dark Light2Dark added the enhancement New feature or request label Apr 13, 2026
@Light2Dark Light2Dark changed the title add reveal.js for slides swap reveal.js instead of swiper for slides Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant