Giter VIP home page Giter VIP logo

Comments (14)

annevk avatar annevk commented on July 3, 2024

I guess it's too late to stop doing that?

@asutherland @mkruisselbrink @jakearchibald

from xhr.

mkruisselbrink avatar mkruisselbrink commented on July 3, 2024

"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.

from xhr.

annevk avatar annevk commented on July 3, 2024

Yeah, removing FileReader. If it's too late I support doing this.

from xhr.

Rashika101 avatar Rashika101 commented on July 3, 2024

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

from xhr.

saschanaz avatar saschanaz commented on July 3, 2024

xhr/xhr.bs

Lines 1764 to 1765 in b7726ee

[Exposed=(Window,DedicatedWorker,SharedWorker)]
interface ProgressEvent : Event {

Currently the line 1764 lacks ServiceWorker, so we need to add one. I don't think we need anything other than that 👍

from xhr.

Rashika101 avatar Rashika101 commented on July 3, 2024

Hey!! I wanna do that will you please assigned this issue to me that will be great .

from xhr.

saschanaz avatar saschanaz commented on July 3, 2024

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.

from xhr.

Rashika101 avatar Rashika101 commented on July 3, 2024

Thank You
@saschanaz
but what is fixed in Fixes #305.
If You don't mind Could You please explain little deeper.

from xhr.

saschanaz avatar saschanaz commented on July 3, 2024

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.

from xhr.

Rashika101 avatar Rashika101 commented on July 3, 2024

Understood...But no Description box is coming.
written -"there isn't nothing to compare"
How to do that?

from xhr.

saschanaz avatar saschanaz commented on July 3, 2024

You committed the fix and you see that message? Weird, could you share the screenshot?

from xhr.

Rashika101 avatar Rashika101 commented on July 3, 2024

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..

from xhr.

saschanaz avatar saschanaz commented on July 3, 2024

No problem! The easiest way:

  1. Open https://github.com/whatwg/xhr/blob/main/xhr.bs
  2. Click the edit button at the right side (which looks like a pen)
  3. Search interface ProgressEvent and add ,ServiceWorker to the previous line
  4. At the "Propose changes" block, write a title e.g. "Expose ProgressEvent to ServiceWorker"
  5. Click "Propose changes" button.
  6. Cilck "Create pull request"
  7. 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.)
  8. Cilck "Create pull request" again
  9. That's it!

I'll happily write steps for GitHub Desktop if you are using it BTW.

from xhr.

Rashika101 avatar Rashika101 commented on July 3, 2024

@saschanaz
Thank You so so much ..
I did that..
What next to do?

from xhr.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.