Expose ProgressEvent to ServiceWorker
saschanaz opened this issue · 14 comments
FileReader is exposed to ServiceWorker and uses ProgressEvent, but the event interface is not exposed to ServiceWorker since 9d0a9bb.
Exposing it to ServiceWorker also matches Gecko and WebKit implementation:
I guess it's too late to stop doing that?
"that" being exposing FileReader to service workers? Now all major browsers have implemented Blob.stream/text/arrayBuffer there at least is an alternative to FileReader (other than wrapping a Blob in a Response). My gut feeling is that actually removing FileReader from service workers would be challenging though.
Yeah, removing FileReader
. If it's too late I support doing this.
Hey!!
Are you willing to accept new contributors? I am interested into contributing to this repository so, if you could explain a bit more this feature request, it would be great.
Thanks
Lines 1764 to 1765 in b7726ee
Currently the line 1764 lacks ServiceWorker
, so we need to add one. I don't think we need anything other than that 👍
Hey!! I wanna do that will you please assigned this issue to me that will be great .
I don't think assigning is used in this repo. Just open a PR and make sure you include Fixes #305
in the description 😉. It will automatically link your PR to this issue.
Thank You
@saschanaz
but what is fixed in Fixes #305.
If You don't mind Could You please explain little deeper.
Hmm, I mean when you open a pull request after writing the actual fix, you'll want to include the "Fixes" text to the content body of the post. It's not a requirement but just a recommendation, so if you have no idea please just proceed.
Understood...But no Description box is coming.
written -"there isn't nothing to compare"
How to do that?
You committed the fix and you see that message? Weird, could you share the screenshot?
Can we please start it again?
Excuse me!! But I just got confused...
How to add a service Worker? I did in my own way...which is not adjusting with the repo.
Please teach me a little more..
No problem! The easiest way:
- Open https://github.com/whatwg/xhr/blob/main/xhr.bs
- Click the edit button at the right side (which looks like a pen)
- Search
interface ProgressEvent
and add,ServiceWorker
to the previous line - At the "Propose changes" block, write a title e.g. "Expose ProgressEvent to ServiceWorker"
- Click "Propose changes" button.
- Cilck "Create pull request"
- Type "x" inside
[ ]
so that it can be[x]
, for "At least two implementers are interested" part, and replace "..." with "Gecko" and "WebKit". This is based on my observation in #305 (comment). (You can skip this.) - Cilck "Create pull request" again
- That's it!
I'll happily write steps for GitHub Desktop if you are using it BTW.
@saschanaz
Thank You so so much ..
I did that..
What next to do?