-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
add turbopack-image #4621
add turbopack-image #4621
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
9 Ignored Deployments
|
✅ This changes can build |
Benchmark for b799c7aClick to view benchmark
|
Benchmark for bcc1a7dClick to view benchmark
|
|
7db9535
to
26ec98a
Compare
image: image::DynamicImage, | ||
format: ImageFormat, | ||
) -> Result<(String, u32, u32)> { | ||
let small_image = image.resize(BLUR_IMG_SIZE, BLUR_IMG_SIZE, FilterType::Triangle); |
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.
So we're not going with Lanczos after all?
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.
I don't think it's worth it for a 8x8 image...
3860e0e
to
1c73cfc
Compare
### What? add support for blur placeholder generation to turbopack add `StructuredImageModuleType` which is used with `ModuleType::Custom` to allow importing an image as `{ url, width, height, blurDataURL, blurWidth, blurHeight }` in contrast to next.js with webpack this will also generate blur placeholder in development instead of using a _next/image reference. This should lead to more production-like experience (at the cost of a little bit of compilation time). turbo PR: vercel/turborepo#4621 ### Why? Turbopack was crashing on `placeholder="blur"` before. fixes WEB-534 ### Turbopack changes * vercel/turborepo#4521 <!-- OJ Kwon - feat(contextcondition): support InPath contextcondition --> * vercel/turborepo#4601 <!-- Alex Kirszenberg - Chunking Context Refactor pt. 3: Address PR comments from pt. 2 --> * vercel/turborepo#4623 <!-- Tobias Koppers - exclude turborepo from turbopack bench tests --> * vercel/turborepo#4399 <!-- Leah - support require.context --> * vercel/turborepo#4610 <!-- OJ Kwon - test(subset): add mdx test into subset --> * vercel/turborepo#4624 <!-- Tobias Koppers - run benchmarks on windows and macOS too --> * vercel/turborepo#4620 <!-- Alex Kirszenberg - Make ContainmentTree fully generic --> * vercel/turborepo#4600 <!-- Tobias Koppers - add getChunkPath method --> * vercel/turborepo#4621 <!-- Tobias Koppers - add turbopack-image --> * vercel/turborepo#4639 <!-- Tobias Koppers - restrict snapshot path for windows path length limit --> * vercel/turborepo#4641 <!-- Tobias Koppers - put webp behind a feature flag -->
### Description allows to have custom plugins for module types add turbopack-image crate which adds some image processing and blur placholder generation next.js PR: vercel/next.js#48531
### Description allows to have custom plugins for module types add turbopack-image crate which adds some image processing and blur placholder generation next.js PR: #48531
### Description allows to have custom plugins for module types add turbopack-image crate which adds some image processing and blur placholder generation next.js PR: #48531
### Description allows to have custom plugins for module types add turbopack-image crate which adds some image processing and blur placholder generation next.js PR: #48531
Description
allows to have custom plugins for module types
add turbopack-image crate which adds some image processing and blur placholder generation
next.js PR: vercel/next.js#48531