-
Notifications
You must be signed in to change notification settings - Fork 211
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
[feature request] Workers support #16
Comments
Yeah! You're right. Do you or someone have any clue on how to do that? I'm trying to make it possible, but the entire plugin is done in jQuery, and worker's need to be in native JavaScript. |
If there is workers implementation I'd definitely use it. Lag is too heavy |
For now lag is caused also by events handled in wrong way. In my fork there is a lot of performance fixes: |
I'll look into your fork, thanks @korpirkor ! |
The blur method you use is significantly faster, but also uglier. I'll study both to see which one suits me better, thanks! |
You can still use StackBlur instead of StackBoxBlur, just revert this commit CezaryDanielNowak@58878ab . I could make switch for blur algo but it would increase file size |
I've finally done it by using Web workers. |
When running blurjs, there is noticeable lag. Using Web Worker to transform image would fix the lag :)
The text was updated successfully, but these errors were encountered: