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

Bump wasm-vips to 0.0.9 #489

Merged
merged 2 commits into from
Jun 2, 2024
Merged

Conversation

kleisauke
Copy link
Contributor

@kleisauke kleisauke commented Jun 1, 2024

- Worker script is inlined into the main output file.
- Simplify resize logic using the `pageHeight` property.

Resolves: swissspidy#474.
@swissspidy
Copy link
Owner

Amazing, thank you! I just saw the notification for the new version and I think both are really nice additions :)

@swissspidy

This comment was marked as resolved.

`workaroundCors` would only work when wasm-vips isn't bundled.
@kleisauke
Copy link
Contributor Author

Ah, perhaps I misunderstood issue #474, as I just noticed that there is also a vips.worker.js file in the build directory.

() => import( /* webpackChunkName: 'vips' */ '@mexp/vips' )

Copy link
Owner

@swissspidy swissspidy left a comment

Choose a reason for hiding this comment

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

Thank you so much!

@swissspidy swissspidy merged commit 5fb06e0 into swissspidy:main Jun 2, 2024
20 checks passed
swissspidy pushed a commit that referenced this pull request Jun 2, 2024
@kleisauke kleisauke deleted the wasm-vips-0.0.9 branch June 2, 2024 15:28
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.

2 participants