react-worker-image

react-worker-image copied to clipboard

How to mock a worker when testing with jest? Should we even test workers?

10

ReferenceError: Worker is not defined

manjula-dube

nitish24p

[Opinionated] Add alt attribute & remove container

5

I've added an alt attribute, because "Placeholder" and "Worker" as `alt` is not really nice. Second thing I've done is remove the container, because if you're using `` as replacement...

kurtextrem

Limit Workers to max. avail. CPUs

4

Fixes #10

kurtextrem

Hey there, Great work on the library, I think most code bases now a days use prettier to maintain code style consistency and readability, even some of the most popular...

soorajvnair

Hey there, Great job on the library, I think it's useful to have onLoad property available, so that we're aware when the image has loaded, in my use-case i want...

soorajvnair

Processing an image with filters supplied as props and return a base64 image.

2

@nitish24p Should we take this up? What are your thoughts on this? Processing an image with filters supplied as props and return a base64 image.

manjula-dube

let user pass extra attributes

5

hosmelq

Config that decides whether to use shared or single worker: (This should be passed as props)

2

manjula-dube

Is spawning a new worker for every image a good idea?

9

See title. I don't really have experience with those, but shouldn't be one worker sufficient? I imagine loading 50+ images via 50 workers is slower than loading 50 images with...

kurtextrem