-
Notifications
You must be signed in to change notification settings - Fork 889
add snappy parameter to speed up auto-export #8032
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
bc5386e to
0fc434a
Compare
frontend/src/utils/html-to-image.ts
Outdated
| "clip-path", | ||
|
|
||
| // Overflow & Visibility | ||
| "overflow", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does it make sense to comment overflow so we don't see scrollbars?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could, however, this path is only triggered on auto-export. Probably have to do the styles injection too.
#8005
|
🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.19.7-dev33 |
📝 Summary
Adds a new 'snappy' parameter to speed up auto-export to ipynb. This makes it so that taking screenshots will only look for necessary css styles (which offers a lot of speedup)
🔍 Description of Changes
📋 Checklist